Fix: Fix SSO setting page permissions

This commit is contained in:
Gustav Hansen 2023-06-23 15:21:16 +02:00
parent dfc130f2f1
commit ee66e7cc95

View File

@ -38,13 +38,18 @@ export const SingleSignOn = () => {
const { formatMessage } = useIntl(); const { formatMessage } = useIntl();
const permissions = useSelector(selectAdminPermissions); const permissions = useSelector(selectAdminPermissions);
// TODO: this is necessary because otherwise we run into an
// infinite rendering loop
const permissionsMemoized = React.useMemo(() => {
return {
...permissions.settings.sso,
readRoles: permissions.settings.roles.read,
};
}, [permissions.settings.roles.read, permissions.settings.sso]);
const { const {
isLoading: isLoadingForPermissions, isLoading: isLoadingForPermissions,
allowedActions: { canUpdate, canReadRoles }, allowedActions: { canUpdate, canReadRoles },
} = useRBAC({ } = useRBAC(permissionsMemoized);
...permissions.settings.sso,
readRoles: permissions.settings.roles.read,
});
const [ const [
{ formErrors, initialData, isLoading, modifiedData, showHeaderButtonLoader }, { formErrors, initialData, isLoading, modifiedData, showHeaderButtonLoader },
@ -251,7 +256,7 @@ const ProtectedSSO = () => {
const permissions = useSelector(selectAdminPermissions); const permissions = useSelector(selectAdminPermissions);
return ( return (
<CheckPagePermissions permissions={permissions.sso.main}> <CheckPagePermissions permissions={permissions.settings.sso.main}>
<SingleSignOn /> <SingleSignOn />
</CheckPagePermissions> </CheckPagePermissions>
); );