feat(create-role): make isCreator condition checked by default (#1276)

packages/web/src/components/ControlledCheckbox/index.tsx

@@ -4,9 +4,12 @@ import Checkbox, { CheckboxProps } from '@mui/material/Checkbox';
 
 type ControlledCheckboxProps = {
   name: string;
-} & CheckboxProps;
+  defaultValue?: boolean;
+} & Omit<CheckboxProps, 'defaultValue'>;
 
-export default function ControlledCheckbox(props: ControlledCheckboxProps): React.ReactElement {
+export default function ControlledCheckbox(
+  props: ControlledCheckboxProps
+): React.ReactElement {
   const { control } = useFormContext();
   const {
     required,
@@ -34,24 +37,25 @@ export default function ControlledCheckbox(props: ControlledCheckboxProps): Reac
           ...field
         },
       }) => {
-       return (
+        return (
-        <Checkbox
+          <Checkbox
-          {...checkboxProps}
+            {...checkboxProps}
-          {...field}
+            {...field}
-          checked={!!value}
+            checked={!!value}
-          name={name}
+            name={name}
-          disabled={disabled}
+            disabled={disabled}
-          onChange={(...args) => {
+            onChange={(...args) => {
-            controllerOnChange(...args);
+              controllerOnChange(...args);
-            onChange?.(...args);
+              onChange?.(...args);
-          }}
+            }}
-          onBlur={(...args) => {
+            onBlur={(...args) => {
-            controllerOnBlur();
+              controllerOnBlur();
-            onBlur?.(...args);
+              onBlur?.(...args);
-          }}
+            }}
-          inputRef={ref}
+            inputRef={ref}
-        />
+          />
-      )}}
+        );
+      }}
     />
   );
 }

packages/web/src/components/PermissionCatalogField/PermissionSettings.ee.tsx

@@ -22,18 +22,14 @@ type PermissionSettingsProps = {
   onClose: () => void;
   fieldPrefix: string;
   subject: string;
+  defaultChecked?: boolean;
   actions: IPermissionCatalog['actions'];
   conditions: IPermissionCatalog['conditions'];
-}
+};
 
 export default function PermissionSettings(props: PermissionSettingsProps) {
-  const {
+  const { onClose, fieldPrefix, subject, actions, conditions, defaultChecked } =
-    onClose,
+    props;
-    fieldPrefix,
-    subject,
-    actions,
-    conditions,
-  } = props;
 
   const formatMessage = useFormatMessage();
   const { getValues, resetField } = useFormContext();
@@ -47,7 +43,7 @@ export default function PermissionSettings(props: PermissionSettingsProps) {
     }
 
     onClose();
-  }
+  };
 
   const apply = () => {
     for (const action of actions) {
@@ -59,13 +55,11 @@ export default function PermissionSettings(props: PermissionSettingsProps) {
     }
 
     onClose();
-  }
+  };
 
   return (
     <Dialog open onClose={cancel}>
-      <DialogTitle>
+      <DialogTitle>{formatMessage('permissionSettings.title')}</DialogTitle>
-        {formatMessage('permissionSettings.title')}
-      </DialogTitle>
 
       <DialogContent>
         <TableContainer component={Paper}>
@@ -74,14 +68,14 @@ export default function PermissionSettings(props: PermissionSettingsProps) {
               <TableRow>
                 <TableCell component="th" />
 
-                {actions.map(action => (
+                {actions.map((action) => (
                   <TableCell component="th" key={action.key}>
                     <Typography
                       variant="subtitle1"
                       align="center"
                       sx={{
                         color: 'text.secondary',
-                        fontWeight: 700
+                        fontWeight: 700,
                       }}
                     >
                       {action.label}
@@ -97,9 +91,7 @@ export default function PermissionSettings(props: PermissionSettingsProps) {
                   sx={{ '&:last-child td': { border: 0 } }}
                 >
                   <TableCell scope="row">
-                    <Typography
+                    <Typography variant="subtitle2">
-                      variant="subtitle2"
-                    >
                       {condition.label}
                     </Typography>
                   </TableCell>
@@ -109,13 +101,16 @@ export default function PermissionSettings(props: PermissionSettingsProps) {
                       key={`${action.key}.${condition.key}`}
                       align="center"
                     >
-                      <Typography
+                      <Typography variant="subtitle2">
-                        variant="subtitle2"
-                      >
                         {action.subjects.includes(subject) && (
                           <ControlledCheckbox
                             name={`${fieldPrefix}.${action.key}.conditions.${condition.key}`}
-                            disabled={getValues(`${fieldPrefix}.${action.key}.value`) !== true}
+                            defaultValue={defaultChecked}
+                            disabled={
+                              getValues(
+                                `${fieldPrefix}.${action.key}.value`
+                              ) !== true
+                            }
                           />
                         )}
 
@@ -131,12 +126,14 @@ export default function PermissionSettings(props: PermissionSettingsProps) {
       </DialogContent>
 
       <DialogActions>
-        <Button onClick={cancel}>{formatMessage('permissionSettings.cancel')}</Button>
+        <Button onClick={cancel}>
+          {formatMessage('permissionSettings.cancel')}
+        </Button>
 
         <Button onClick={apply} color="error">
           {formatMessage('permissionSettings.apply')}
         </Button>
       </DialogActions>
     </Dialog>
-  )
+  );
-}
+}

packages/web/src/components/PermissionCatalogField/index.ee.tsx

@@ -19,11 +19,13 @@ import PermissionCatalogFieldLoader from './PermissionCatalogFieldLoader';
 type PermissionCatalogFieldProps = {
   name?: string;
   disabled?: boolean;
+  defaultChecked?: boolean;
 };
 
 const PermissionCatalogField = ({
   name = 'permissions',
   disabled = false,
+  defaultChecked = false,
 }: PermissionCatalogFieldProps) => {
   const { permissionCatalog, loading } = usePermissionCatalog();
   const [dialogName, setDialogName] = React.useState<string>();
@@ -98,6 +100,7 @@ const PermissionCatalogField = ({
                       subject={subject.key}
                       actions={permissionCatalog.actions}
                       conditions={permissionCatalog.conditions}
+                      defaultChecked={defaultChecked}
                     />
                   )}
                 </Stack>

packages/web/src/pages/CreateRole/index.ee.tsx

@@ -74,7 +74,10 @@ export default function CreateRole(): React.ReactElement {
                 fullWidth
               />
 
-              <PermissionCatalogField name="computedPermissions" />
+              <PermissionCatalogField
+                name="computedPermissions"
+                defaultChecked={true}
+              />
 
               <LoadingButton
                 type="submit"