Improve MCC label format

Author: bbimber

mcc/src/client/RequestReview/components/RabReviewForm.tsx

@@ -124,8 +124,8 @@ export default function RabReviewForm(props: {requestId: string}) {
             <TableRow>
                 <TableCell>
                     <FormControl>
-                        <InputLabel id={"review-input-label"}>Enter review...</InputLabel>
-                        <Select id={"review"} variant={"outlined"} name={"review"} aria-label="Review" label={"Review"} error={hasSubmitted && !recordData[0].review} onChange={handleChange} required={true} value={recordData[0].review ?? ''} fullWidth={true} displayEmpty={true}>
+                        <InputLabel id={"review-input-label"}>Review</InputLabel>
+                        <Select id={"review"} variant={"outlined"} name={"review"} aria-label="Review" label={"Review"} labelId={"review-input-label"} error={hasSubmitted && !recordData[0].review} onChange={handleChange} required={true} value={recordData[0].review ?? ''} fullWidth={true} displayEmpty={true}>
                             <MenuItem value={""}>Not Decided</MenuItem>
                             <MenuItem value={"I recommend this proposal"}>I recommend this proposal</MenuItem>
                             <MenuItem value={"I recommend this proposal with conditions"}>I recommend this proposal with conditions</MenuItem>

mcc/src/org/labkey/mcc/MccManager.java

@@ -56,10 +56,10 @@ public enum RequestStatus
         Fulfilled(8, "Fulfilled", MccRequestAdminPermission.class),
         Withdrawn(9, "Withdrawn", MccRequestorPermission.class);
 
-        int sortOrder;
-        String label;
-        Class<? extends Permission> updatePermission;
-        Class<? extends Permission> insertPermission;
+        final int sortOrder;
+        final String label;
+        final Class<? extends Permission> updatePermission;
+        final Class<? extends Permission> insertPermission;
 
         RequestStatus(int sortOrder, String label, Class<? extends Permission> editPermission)
         {