🗑️(frontend) clean member feature

Clean a bit member feature, remove unused files
and refactor some code.

Author: AntoLC

src/frontend/apps/impress/src/components/index.ts

@@ -5,5 +5,6 @@ export * from './DropButton';
 export * from './Icon';
 export * from './InfiniteScroll';
 export * from './Link';
+export * from './SideModal';
 export * from './Text';
 export * from './TextErrors';

src/frontend/apps/impress/src/features/docs/members/invitation-list/api/index.ts

@@ -1,3 +1,4 @@
+export * from './useCreateDocInvitation';
 export * from './useDeleteDocInvitation';
 export * from './useDocInvitations';
 export * from './useUpdateDocInvitation';

src/frontend/apps/impress/src/features/docs/members/members-add/api/useCreateDocInvitation.tsx renamed to src/frontend/apps/impress/src/features/docs/members/invitation-list/api/useCreateDocInvitation.tsx

@@ -3,10 +3,12 @@ import { useMutation, useQueryClient } from '@tanstack/react-query';
 import { APIError, errorCauses, fetchAPI } from '@/api';
 import { User } from '@/core/auth';
 import { Doc, Role } from '@/features/docs/doc-management';
+import { OptionType } from '@/features/docs/members/members-add/types';
 import { ContentLanguage } from '@/i18n/types';
 
-import { KEY_LIST_DOC_INVITATIONS } from '../../invitation-list';
-import { DocInvitation, OptionType } from '../types';
+import { Invitation } from '../types';
+
+import { KEY_LIST_DOC_INVITATIONS } from './useDocInvitations';
 
 interface CreateDocInvitationParams {
   email: User['email'];
@@ -20,7 +22,7 @@ export const createDocInvitation = async ({
   role,
   docId,
   contentLanguage,
-}: CreateDocInvitationParams): Promise<DocInvitation> => {
+}: CreateDocInvitationParams): Promise<Invitation> => {
   const response = await fetchAPI(`documents/${docId}/invitations/`, {
     method: 'POST',
     headers: {
@@ -42,12 +44,12 @@ export const createDocInvitation = async ({
     );
   }
 
-  return response.json() as Promise<DocInvitation>;
+  return response.json() as Promise<Invitation>;
 };
 
-export function useCreateInvitation() {
+export function useCreateDocInvitation() {
   const queryClient = useQueryClient();
-  return useMutation<DocInvitation, APIError, CreateDocInvitationParams>({
+  return useMutation<Invitation, APIError, CreateDocInvitationParams>({
     mutationFn: createDocInvitation,
     onSuccess: () => {
       void queryClient.invalidateQueries({

src/frontend/apps/impress/src/features/docs/members/members-add/api/index.ts

@@ -1,3 +1,2 @@
-export * from './useCreateDocInvitation';
 export * from './useCreateDocAccess';
 export * from './useUsers';

src/frontend/apps/impress/src/features/docs/members/members-add/components/AddMembers.tsx

@@ -9,9 +9,10 @@ import { useTranslation } from 'react-i18next';
 import { APIError } from '@/api';
 import { Box, Card, IconBG } from '@/components';
 import { Doc, Role } from '@/features/docs/doc-management';
+import { useCreateDocInvitation } from '@/features/docs/members/invitation-list/';
 import { useLanguage } from '@/i18n/hooks/useLanguage';
 
-import { useCreateDocAccess, useCreateInvitation } from '../api';
+import { useCreateDocAccess } from '../api';
 import {
   OptionInvitation,
   OptionNewMember,
@@ -39,7 +40,7 @@ export const AddMembers = ({ currentRole, doc }: ModalAddMembersProps) => {
   const [selectedUsers, setSelectedUsers] = useState<OptionsSelect>([]);
   const [selectedRole, setSelectedRole] = useState<Role>();
   const { toast } = useToastProvider();
-  const { mutateAsync: createInvitation } = useCreateInvitation();
+  const { mutateAsync: createInvitation } = useCreateDocInvitation();
   const { mutateAsync: createDocAccess } = useCreateDocAccess();
   const [resetKey, setResetKey] = useState(1);
 

src/frontend/apps/impress/src/features/docs/members/members-add/types.tsx

@@ -1,5 +1,4 @@
 import { User } from '@/core/auth';
-import { Doc, Role } from '@/features/docs/doc-management';
 
 export enum OptionType {
   INVITATION = 'invitation',
@@ -25,12 +24,3 @@ export interface OptionNewMember {
 }
 
 export type OptionSelect = OptionNewMember | OptionInvitation;
-
-export interface DocInvitation {
-  id: string;
-  created_at: string;
-  email: string;
-  team: Doc['id'];
-  role: Role;
-  issuer: User['id'];
-}

src/frontend/apps/impress/src/features/docs/members/members-list/assets/icon-remove-member.svg

@@ -70,49 +70,49 @@ export const MemberItem = ({
 
   return (
     <Box $width="100%">
-      <Box
-        $align="center"
-        $direction="row"
-        $gap="1rem"
-        $justify="space-between"
-        $width="100%"
-        $wrap="wrap"
-      >
-        <Box $direction="row" $gap="1rem">
-          <IconBG iconName="account_circle" $size="2rem" />
+      <Box $direction="row" $gap="1rem">
+        <IconBG iconName="account_circle" $size="2rem" />
+        <Box
+          $align="center"
+          $direction="row"
+          $gap="1rem"
+          $justify="space-between"
+          $width="100%"
+          $wrap="wrap"
+        >
           <Text $justify="center">{access.user.email}</Text>
-        </Box>
-        <Box $direction="row" $gap="1rem" $align="center">
-          <Box $minWidth="13rem">
-            <ChooseRole
-              label={t('Role')}
-              defaultRole={localRole}
-              currentRole={currentRole}
+          <Box $direction="row" $gap="1rem" $align="center">
+            <Box $minWidth="13rem">
+              <ChooseRole
+                label={t('Role')}
+                defaultRole={localRole}
+                currentRole={currentRole}
+                disabled={isNotAllowed}
+                setRole={(role) => {
+                  setLocalRole(role);
+                  updateDocAccess({
+                    docId,
+                    accessId: access.id,
+                    role,
+                  });
+                }}
+              />
+            </Box>
+            <Button
+              color="tertiary-text"
+              icon={
+                <Text
+                  $isMaterialIcon
+                  $theme={isNotAllowed ? 'greyscale' : 'primary'}
+                  $variation={isNotAllowed ? '500' : 'text'}
+                >
+                  delete
+                </Text>
+              }
               disabled={isNotAllowed}
-              setRole={(role) => {
-                setLocalRole(role);
-                updateDocAccess({
-                  docId,
-                  accessId: access.id,
-                  role,
-                });
-              }}
+              onClick={() => removeDocAccess({ docId, accessId: access.id })}
             />
           </Box>
-          <Button
-            color="tertiary-text"
-            icon={
-              <Text
-                $isMaterialIcon
-                $theme={isNotAllowed ? 'greyscale' : 'primary'}
-                $variation={isNotAllowed ? '500' : 'text'}
-              >
-                delete
-              </Text>
-            }
-            disabled={isNotAllowed}
-            onClick={() => removeDocAccess({ docId, accessId: access.id })}
-          />
         </Box>
       </Box>
       {(errorUpdate || errorDelete) && (