Merge pull request #981 from Northeastern-Electric-Racing/#980-dont-r…

…equire-notes #980 Remove Requirement for notes/assignees
Northeastern-Electric-Racing · Mar 15, 2023 · a71c05a · a71c05a
2 parents 9b649ef + 6d06155
commit a71c05a
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/src/frontend/src/pages/ProjectDetailPage/ProjectViewContainer/TaskListNotesModal.tsx b/src/frontend/src/pages/ProjectDetailPage/ProjectViewContainer/TaskListNotesModal.tsx
@@ -53,10 +53,10 @@ export interface FormInput {
 }
 
 const schema = yup.object().shape({
-  notes: yup.string().required(),
+  notes: yup.string(),
   deadline: yup.date().required(),
   priority: yup.string().required(),
-  assignees: yup.array().required(),
+  assignees: yup.array(),
   title: yup.string().required()
 });
 
@@ -260,7 +260,6 @@ const TaskListNotesModal: React.FC<TaskListNotesModalProps> = ({
                   <Controller
                     name="assignees"
                     control={control}
-                    rules={{ required: true }}
                     render={({ field: { onChange, value } }) => (
                       <Autocomplete
                         isOptionEqualToValue={(option, value) => option.id === value.id}
@@ -304,10 +303,8 @@ const TaskListNotesModal: React.FC<TaskListNotesModalProps> = ({
                   <Controller
                     name={'notes'}
                     control={control}
-                    rules={{ required: true }}
                     render={({ field: { onChange, value } }) => (
                       <TextField
-                        required
                         onChange={onChange}
                         value={value}
                         multiline