Fixed optional photo uploading

Author: dburkhart07

apps/backend/src/foodRequests/request.controller.ts

@@ -125,18 +125,15 @@ export class FoodRequestsController {
   async confirmDelivery(
     @Param('requestId', ParseIntPipe) requestId: number,
     @Body() body: { dateReceived: string; feedback: string },
-    @UploadedFiles() photos: Express.Multer.File[],
+    @UploadedFiles() photos?: Express.Multer.File[],
   ): Promise<FoodRequest> {
-    if (!photos || photos.length === 0) {
-      throw new Error('No photos uploaded');
-    }
-
     const formattedDate = new Date(body.dateReceived);
     if (isNaN(formattedDate.getTime())) {
       throw new Error('Invalid date format for deliveryDate');
     }
 
-    const uploadedPhotoUrls = await this.awsS3Service.upload(photos);
+    const uploadedPhotoUrls =
+      photos && photos.length > 0 ? await this.awsS3Service.upload(photos) : [];
 
     return this.requestsService.updateDeliveryDetails(
       requestId,

apps/frontend/src/components/forms/deliveryConfirmationModalButton.tsx

@@ -149,9 +149,11 @@ export const submitDeliveryConfirmationFormModal: ActionFunction = async ({
 
   confirmDeliveryData.append('feedback', form.get('feedback') as string);
 
-  globalPhotos.forEach((photo) => {
-    confirmDeliveryData.append('photos', photo);
-  });
+  if (globalPhotos.length > 0) {
+    globalPhotos.forEach((photo) => {
+      confirmDeliveryData.append('photos', photo);
+    });
+  }
 
   try {
     const response = await fetch(