Skip to content

Commit c78f810

Browse files
committed
Revert "#1195 Cleaned up default values"
This reverts commit c80f405.
1 parent c80f405 commit c78f810

3 files changed

Lines changed: 37 additions & 46 deletions

File tree

src/frontend/src/components/ReactHookEditableList.tsx

Lines changed: 12 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -1,46 +1,36 @@
11
import { Grid, Button, IconButton, TextField } from '@mui/material';
22
import DeleteIcon from '@mui/icons-material/Delete';
33
import { FieldArrayWithId, UseFieldArrayRemove, UseFormRegister, UseFieldArrayAppend } from 'react-hook-form';
4-
import { Box } from '@mui/system';
54

65
interface ReactHookEditableListProps {
76
name: string;
87
ls: FieldArrayWithId[];
98
register: UseFormRegister<any>;
109
append: UseFieldArrayAppend<any, any>;
1110
remove: UseFieldArrayRemove;
12-
addMessage: string;
1311
}
1412

15-
const ReactHookEditableList: React.FC<ReactHookEditableListProps> = ({
16-
name,
17-
ls,
18-
register,
19-
append,
20-
remove,
21-
addMessage = '+ ADD NEW BULLET'
22-
}) => {
13+
const ReactHookEditableList: React.FC<ReactHookEditableListProps> = ({ name, ls, register, append, remove }) => {
2314
return (
2415
<>
25-
<Button
26-
variant="contained"
27-
color="primary"
28-
onClick={() => append({ bulletId: -1, detail: '' })}
29-
sx={{ my: 2, width: 'max-content' }}
30-
>
31-
{addMessage}
32-
</Button>
3316
{ls.map((_element, i) => {
3417
return (
35-
//<Grid item sx={{ display: 'grid', alignItems: 'center' }}>
36-
<Box sx={{ width: 10 / 10 }}>
37-
<TextField required autoComplete="off" sx={{ width: 2 / 10 }} {...register(`${name}.${i}.detail`)} />
18+
<Grid item sx={{ display: 'flex', alignItems: 'center' }}>
19+
<TextField required autoComplete="off" sx={{ width: 9 / 10 }} {...register(`${name}.${i}.detail`)} />
3820
<IconButton type="button" onClick={() => remove(i)} sx={{ mx: 1, my: 0 }}>
3921
<DeleteIcon />
4022
</IconButton>
41-
</Box>
23+
</Grid>
4224
);
4325
})}
26+
<Button
27+
variant="contained"
28+
color="success"
29+
onClick={() => append({ bulletId: -1, detail: '' })}
30+
sx={{ my: 2, width: 'max-content' }}
31+
>
32+
+ Add New Bullet
33+
</Button>
4434
</>
4535
);
4636
};

src/frontend/src/pages/ProjectDetailPage/ProjectEdit/ProjectEditContainer.tsx

Lines changed: 14 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -214,14 +214,7 @@ const ProjectEditContainer: React.FC<ProjectEditContainerProps> = ({ project, ex
214214
<LinksEditView watch={watch} ls={links} register={register} append={appendLink} remove={removeLink} />
215215
</PageBlock>
216216
<PageBlock title="Goals">
217-
<ReactHookEditableList
218-
name="goals"
219-
register={register}
220-
ls={goals}
221-
append={appendGoal}
222-
remove={removeGoal}
223-
addMessage="Add Goal"
224-
/>
217+
<ReactHookEditableList name="goals" register={register} ls={goals} append={appendGoal} remove={removeGoal} />
225218
</PageBlock>
226219
<PageBlock title="Features">
227220
<ReactHookEditableList
@@ -230,7 +223,6 @@ const ProjectEditContainer: React.FC<ProjectEditContainerProps> = ({ project, ex
230223
ls={features}
231224
append={appendFeature}
232225
remove={removeFeature}
233-
addMessage="Add Feature"
234226
/>
235227
</PageBlock>
236228
<PageBlock title="Other Constraints">
@@ -240,18 +232,22 @@ const ProjectEditContainer: React.FC<ProjectEditContainerProps> = ({ project, ex
240232
ls={constraints}
241233
append={appendConstraint}
242234
remove={removeConstraint}
243-
addMessage="Add Constraint"
244235
/>
245236
</PageBlock>
246237
<PageBlock title="Rules">
247-
<ReactHookEditableList
248-
name="rules"
249-
register={register}
250-
ls={rules}
251-
append={appendRule}
252-
remove={removeRule}
253-
addMessage="Add Rule"
254-
/>
238+
{rules.map((_rule, i) => {
239+
return (
240+
<Grid item sx={{ display: 'flex', alignItems: 'center' }}>
241+
<TextField required autoComplete="off" {...register(`rules.${i}.rule`)} sx={{ width: 5 / 10 }} />
242+
<IconButton type="button" onClick={() => removeRule(i)} sx={{ mx: 1, my: 0 }}>
243+
<DeleteIcon />
244+
</IconButton>
245+
</Grid>
246+
);
247+
})}
248+
<Button variant="contained" color="success" onClick={() => appendRule({ rule: '' })} sx={{ mt: 2 }}>
249+
+ ADD NEW RULE
250+
</Button>
255251
</PageBlock>
256252
<Box textAlign="right" sx={{ my: 2 }}>
257253
<NERFailButton variant="contained" onClick={exitEditMode} sx={{ mx: 1 }}>

src/frontend/src/pages/WorkPackageDetailPage/ActivateWorkPackageModalContainer/ActivateWorkPackageModal.tsx

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,8 @@ const schema = yup.object().shape({
4747
confirmDetails: yup.boolean().required()
4848
});
4949

50+
//startDate: new Date().toLocaleDateString(),
51+
5052
const ActivateWorkPackageModal: React.FC<ActivateWorkPackageModalProps> = ({
5153
allUsers,
5254
wbsNum,
@@ -60,16 +62,19 @@ const ActivateWorkPackageModal: React.FC<ActivateWorkPackageModalProps> = ({
6062
const daysUntilNextMonday = (7 - today + 1) % 7;
6163
startDate.setDate(startDate.getDate() + daysUntilNextMonday);
6264
}
65+
const { reset, handleSubmit, control } = useForm<FormInput>({
66+
resolver: yupResolver(schema),
67+
defaultValues: {
68+
confirmDetails: false,
69+
startDate
70+
}
71+
});
6372

6473
const defaultValues: FormInput = {
65-
startDate: startDate,
74+
startDate: new Date(),
6675
confirmDetails: false
6776
};
6877

69-
const { reset, handleSubmit, control } = useForm<FormInput>({
70-
resolver: yupResolver(schema)
71-
});
72-
7378
const [projectLeadId, setProjectLeadId] = useState<string>();
7479
const [projectManagerId, setProjectManagerId] = useState<string>();
7580
const toast = useToast();
@@ -144,7 +149,7 @@ const ActivateWorkPackageModal: React.FC<ActivateWorkPackageModalProps> = ({
144149
inputFormat="yyyy-MM-dd"
145150
onChange={(date) => onChange(date ?? new Date())}
146151
className={'padding: 10'}
147-
value={startDate}
152+
value={value}
148153
shouldDisableDate={disableStartDate}
149154
renderInput={(params) => <TextField autoComplete="off" {...params} />}
150155
/>

0 commit comments

Comments
 (0)