Merge pull request #4145 from Northeastern-Electric-Racing/#4101-sponsor-logo-image-id

#4101 sponsor logo image id in backend

Author: wavehassman

src/backend/src/controllers/finance.controllers.ts

@@ -43,7 +43,8 @@ export default class FinanceController {
         contactPhone,
         contactPosition,
         stockDescription,
-        discountDescription
+        discountDescription,
+        req.file
       );
       res.status(200).json(sponsor);
     } catch (error: unknown) {
@@ -383,7 +384,8 @@ export default class FinanceController {
         contactPhone,
         contactPosition,
         stockDescription,
-        discountDescription
+        discountDescription,
+        req.file
       );
 
       res.status(200).json(updatedSponsor);

src/backend/src/routes/finance.routes.ts

@@ -9,11 +9,15 @@ import {
 } from '../utils/validation.utils.js';
 import { body } from 'express-validator';
 import FinanceController from '../controllers/finance.controllers.js';
+import multer, { memoryStorage } from 'multer';
+import { MAX_FILE_SIZE } from 'shared';
 
 const financeRouter = express.Router();
+const upload = multer({ limits: { fileSize: MAX_FILE_SIZE }, storage: memoryStorage() });
 
 financeRouter.post(
   '/sponsor/create',
+  upload.single('logoImage'),
   nonEmptyString(body('name')),
   body('activeStatus').isBoolean(),
   body('valueTypes').isArray(),
@@ -142,6 +146,7 @@ financeRouter.get('/sponsorTiers', FinanceController.getAllSponsorTiers);
 
 financeRouter.post(
   '/sponsor/:sponsorId/edit',
+  upload.single('logoImage'),
   nonEmptyString(body('name')),
   body('activeStatus').isBoolean(),
   body('valueTypes').isArray(),

src/backend/src/services/finance.services.ts

@@ -34,6 +34,7 @@ import {
   getReimbursementRequestWhereInput
 } from '../utils/finance.utils.js';
 import { notifySponsorTaskAssignee } from '../utils/slack.utils.js';
+import { uploadFile } from '../utils/google-integration.utils.js';
 import { isUserFinanceTeamOrHead } from '../utils/reimbursement-requests.utils.js';
 
 export default class FinanceServices {
@@ -56,7 +57,7 @@ export default class FinanceServices {
    * @param contactPosition The position of the sponsor contact.
    * @param sponsorTasks An array of sponsor tasks associated with the sponsor.
    * @param organization The organization for which the sponsor is being created.
-   *
+   * @param logoImage An optional logo image file for the sponsor.
    * @returns The created sponsor object, including associated tasks.
    *
    * @throws AccessDeniedAdminOnlyException If the submitter does not have permission to create a sponsor.
@@ -80,7 +81,8 @@ export default class FinanceServices {
     contactPhone?: string,
     contactPosition?: string,
     stockDescription?: string,
-    discountDescription?: string
+    discountDescription?: string,
+    logoImage?: Express.Multer.File
   ) {
     if (!(await userHasPermission(submitter.userId, organization.organizationId, isHead)))
       throw new AccessDeniedException('Only heads can create a sponsor');
@@ -104,6 +106,8 @@ export default class FinanceServices {
       data: { name: contactName, email: contactEmail, phone: contactPhone, position: contactPosition }
     });
 
+    const { id: logoImageId } = logoImage ? await uploadFile(logoImage) : { id: undefined };
+
     const sponsor = await prisma.sponsor.create({
       data: {
         name,
@@ -118,6 +122,7 @@ export default class FinanceServices {
         taxExempt,
         discountCode,
         sponsorNotes,
+        logoImageId,
         contactId: contact.sponsorContactId,
         sponsorTasks: {
           create: sponsorTasks.map((task) => ({
@@ -1200,6 +1205,8 @@ export default class FinanceServices {
    * @param contactPosition The position of the sponsor contact.
    * @param sponsorTasks An array of sponsor tasks associated with the sponsor.
    * @param organization The organization for which the sponsor is being edited.
+   * @param logoImage An optional logo image file for the sponsor.
+   *
    * @returns the edited sponsor.
    */
 
@@ -1223,7 +1230,8 @@ export default class FinanceServices {
     contactPhone?: string,
     contactPosition?: string,
     stockDescription?: string,
-    discountDescription?: string
+    discountDescription?: string,
+    logoImage?: Express.Multer.File
   ): Promise<Sponsor> {
     if (!(await userHasPermission(submitter.userId, organization.organizationId, isHead)))
       throw new AccessDeniedException('Only heads can edit sponsors.');
@@ -1321,6 +1329,8 @@ export default class FinanceServices {
       data: { name: contactName, email: contactEmail, phone: contactPhone, position: contactPosition }
     });
 
+    const { id: logoImageId } = logoImage ? await uploadFile(logoImage) : { id: undefined };
+
     const updatedSponsor = await prisma.sponsor.update({
       where: { sponsorId: oldSponsor.sponsorId },
       data: {
@@ -1335,7 +1345,8 @@ export default class FinanceServices {
         tier: sponsorTierId ? { connect: { sponsorTierId } } : { disconnect: true },
         taxExempt,
         discountCode,
-        sponsorNotes
+        sponsorNotes,
+        ...(logoImageId && { logoImageId })
       },
       ...getSponsorQueryArgs(organization.organizationId)
     });

src/backend/src/transformers/finance.transformer.ts

@@ -26,6 +26,7 @@ export const sponsorTransformer = (sponsor: Prisma.SponsorGetPayload<SponsorQuer
       : undefined,
     sponsorNotes: sponsor.sponsorNotes ?? undefined,
     discountCode: sponsor.discountCode ?? undefined,
+    logoImageId: sponsor.logoImageId ?? undefined,
     sponsorTasks: sponsor.sponsorTasks.map(sponsorTaskTranformer)
   };
 };

src/shared/src/types/finance-types.ts

@@ -44,6 +44,7 @@ export interface Sponsor {
   sponsorNotes?: string;
   sponsorTasks: SponsorTask[];
   tier?: SponsorTier;
+  logoImageId?: string;
 }
 
 export interface SponsorTask {