Skip to content

Commit 97d0a21

Browse files
committed
#1329 removed lead/manager fields on create and changed cr dropdown
1 parent 60501b3 commit 97d0a21

3 files changed

Lines changed: 37 additions & 26 deletions

File tree

src/frontend/src/components/ChangeRequestDropdown.tsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,10 @@ const ChangeRequestDropdown = ({ control, name, errors, changeHeight = 1 }: Chan
4949
value: cr.crId
5050
}));
5151

52+
const renderValues = new Map<number, string>();
53+
54+
changeRequests.forEach((cr) => renderValues.set(cr.crId, `${cr.crId} - ${wbsPipe(cr.wbsNum)}`));
55+
5256
return (
5357
<Box>
5458
<FormControl fullWidth>
@@ -60,7 +64,7 @@ const ChangeRequestDropdown = ({ control, name, errors, changeHeight = 1 }: Chan
6064
<Select
6165
id="cr-autocomplete"
6266
displayEmpty
63-
renderValue={(value) => value}
67+
renderValue={(value) => renderValues.get(value)}
6468
value={value}
6569
onChange={(event: SelectChangeEvent<number>) => onChange(event.target.value)}
6670
size={'small'}

src/frontend/src/pages/WorkPackageForm/WorkPackageFormDetails.tsx

Lines changed: 31 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ interface Props {
2323
usersForProjectManager: User[];
2424
control: Control<WorkPackageFormViewPayload>;
2525
errors: Partial<FieldErrorsImpl<WorkPackageFormViewPayload>>;
26+
createForm?: boolean;
2627
}
2728

2829
const WorkPackageFormDetails: React.FC<Props> = ({
@@ -33,7 +34,8 @@ const WorkPackageFormDetails: React.FC<Props> = ({
3334
usersForProjectLead,
3435
usersForProjectManager,
3536
control,
36-
errors
37+
errors,
38+
createForm
3739
}) => {
3840
const userToOption = (user?: User): { label: string; id: string } => {
3941
if (!user) return { label: '', id: '' };
@@ -121,30 +123,34 @@ const WorkPackageFormDetails: React.FC<Props> = ({
121123
/>
122124
</FormControl>
123125
</Grid>
124-
<Grid item xs={12} md={5}>
125-
<FormLabel> Project Lead</FormLabel>
126-
<NERAutocomplete
127-
sx={{ width: '100%' }}
128-
id="project-lead-autocomplete"
129-
onChange={(_event, value) => setLead(value?.id)}
130-
options={usersForProjectLead.map(userToOption)}
131-
size="small"
132-
placeholder="Select a Project Lead"
133-
value={userToOption(usersForProjectLead.find((user) => user.userId.toString() === lead))}
134-
/>
135-
</Grid>
136-
<Grid item xs={12} md={5}>
137-
<FormLabel>Project Manager</FormLabel>
138-
<NERAutocomplete
139-
sx={{ width: '100%' }}
140-
id="project-manager-autocomplete"
141-
onChange={(_event, value) => setManager(value?.id)}
142-
options={usersForProjectManager.map(userToOption)}
143-
size="small"
144-
placeholder="Select a Project Manager"
145-
value={userToOption(usersForProjectManager.find((user) => user.userId.toString() === manager))}
146-
/>
147-
</Grid>
126+
{!createForm && (
127+
<>
128+
<Grid item xs={12} md={5}>
129+
<FormLabel> Project Lead</FormLabel>
130+
<NERAutocomplete
131+
sx={{ width: '100%' }}
132+
id="project-lead-autocomplete"
133+
onChange={(_event, value) => setLead(value?.id)}
134+
options={usersForProjectLead.map(userToOption)}
135+
size="small"
136+
placeholder="Select a Project Lead"
137+
value={userToOption(usersForProjectLead.find((user) => user.userId.toString() === lead))}
138+
/>
139+
</Grid>
140+
<Grid item xs={12} md={5}>
141+
<FormLabel>Project Manager</FormLabel>
142+
<NERAutocomplete
143+
sx={{ width: '100%' }}
144+
id="project-manager-autocomplete"
145+
onChange={(_event, value) => setManager(value?.id)}
146+
options={usersForProjectManager.map(userToOption)}
147+
size="small"
148+
placeholder="Select a Project Manager"
149+
value={userToOption(usersForProjectManager.find((user) => user.userId.toString() === manager))}
150+
/>
151+
</Grid>
152+
</>
153+
)}
148154
</Grid>
149155
</Box>
150156
);

src/frontend/src/pages/WorkPackageForm/WorkPackageFormView.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -201,6 +201,7 @@ const WorkPackageFormView: React.FC<WorkPackageFormViewProps> = ({
201201
manager={managerId}
202202
setLead={setLeadId}
203203
setManager={setManagerId}
204+
createForm={createForm}
204205
/>
205206
<Box my={2}>
206207
<Typography variant="h5">Blocked By</Typography>

0 commit comments

Comments
 (0)