refactor: update behaviour depending on user permissions

Author: dcoa

src/authz-module/libraries-manager/LibrariesTeamManager.tsx

@@ -47,10 +47,9 @@ const LibrariesTeamManager = () => {
         pageTitle={pageTitle}
         pageSubtitle={libraryId}
         actions={
-          canManageTeam
-            ? [<PublicReadToggle libraryId={libraryId} key="allow-public-read" />,
-              <AddNewTeamMemberTrigger libraryId={libraryId} key="add-new-member" />]
-            : []
+          [<PublicReadToggle libraryId={libraryId} canEditToggle={canManageTeam} key="allow-public-read" />,
+          ...(canManageTeam ? [<AddNewTeamMemberTrigger libraryId={libraryId} key="add-new-member" />]: [])
+          ]
         }
       >
         <Tabs

src/authz-module/libraries-manager/components/PublicReadToggle.tsx

@@ -6,21 +6,29 @@ import messages from './messages';
 
 type PublicReadToggleProps = {
   libraryId: string;
+  canEditToggle: boolean;
 };
 
-const PublicReadToggle = ({ libraryId }:PublicReadToggleProps) => {
+const PublicReadToggle = ({ libraryId, canEditToggle }: PublicReadToggleProps) => {
   const intl = useIntl();
   const { data: library } = useLibrary(libraryId);
-  const { mutate: updateLibrary } = useUpdateLibrary();
+  const { mutate: updateLibrary, isPending } = useUpdateLibrary();
 
   const onChangeToggle = () => updateLibrary({
     libraryId,
     updatedData: { allowPublicRead: !library.allowPublicRead },
   });
 
+  if (!library) return null;
+
+  if (!library.allowPublicRead && !canEditToggle) {
+    return null;
+  }
+
   return (
     <Form.Switch
       checked={library.allowPublicRead}
+      disabled={!canEditToggle || isPending}
       onChange={onChangeToggle}
       helperText={
         <span>{intl.formatMessage(messages['libraries.authz.public.read.toggle.subtext'])}</span>