Fix validation for edit meal request modal

uwblueprint · Aug 23, 2024 · a316bb6 · a316bb6
1 parent b7657db
commit a316bb6
Showing 1 changed file with 8 additions and 16 deletions.
diff --git a/frontend/src/pages/EditMealRequestForm.tsx b/frontend/src/pages/EditMealRequestForm.tsx
@@ -290,13 +290,6 @@ const EditMealRequestForm = ({
       }
     }
 
-    if (!isEditDonation) {
-      if (deliveryInstructions === "") {
-        setAttemptedSubmit(true);
-        return false;
-      }
-    }
-
     setAttemptedSubmit(false);
     return true;
   };
@@ -527,16 +520,17 @@ const EditMealRequestForm = ({
               </Text>
 
               {!isUpcoming ? (
-                <FormControl mt={3} mb={6} isRequired>
+                <FormControl
+                  mt={3}
+                  mb={6}
+                  isRequired
+                  isInvalid={attemptedSubmit && numberOfMeals <= 0}
+                >
                   <FormLabel
                     variant={{
                       base: "mobile-form-label-bold",
                       md: "form-label-bold",
                     }}
-
-                    // TODO: Hook this up to a state variable
-                    // TODO: Setup correct validation for this
-                    // isInvalid={attemptedSubmit && }
                   >
                     Number of meals
                   </FormLabel>
@@ -552,7 +546,7 @@ const EditMealRequestForm = ({
               ) : null}
 
               {!isUpcoming ? (
-                <FormControl mt={3} mb={6} isRequired>
+                <FormControl mt={3} mb={6}>
                   <FormLabel
                     variant={{
                       base: "mobile-form-label-bold",
@@ -569,14 +563,12 @@ const EditMealRequestForm = ({
                 </FormControl>
               ) : null}
 
-              <FormControl mt={3} mb={6} isRequired>
+              <FormControl mt={3} mb={6}>
                 <FormLabel
                   variant={{
                     base: "mobile-form-label-bold",
                     md: "form-label-bold",
                   }}
-                  // TODO: Setup correct validation for this
-                  // isInvalid={attemptedSubmit && }
                 >
                   Delivery Notes
                 </FormLabel>