cozy-sharing 31.0.2 → 31.2.0

package/src/components/FederatedFolder/FederatedFolderModal.jsx

@@ -1,20 +1,15 @@
 import PropTypes from 'prop-types'
-import React, { useState, useEffect } from 'react'
+import React, { useEffect, useState } from 'react'
 
 import { useClient } from 'cozy-client'
 import { useAlert } from 'cozy-ui/transpiled/react/providers/Alert'
 import { useI18n } from 'twake-i18n'
 
 import { DumbFederatedFolderModal } from './DumbFederatedFolderModal'
+import { getOrCreateFromArray } from '../../helpers/contacts'
 import withLocales from '../../hoc/withLocales'
+import { usePendingRecipients } from '../../hooks/usePendingRecipients'
 import { useSharingContext } from '../../hooks/useSharingContext'
-import {
-  formatRecipients,
-  mergeAndDeduplicateRecipients,
-  moveRecipientToReadWrite,
-  moveRecipientToReadOnly,
-  RECIPIENT_INDEX_PREFIX
-} from '../SharedDrive/helpers'
 
 export const FederatedFolderModal = withLocales(
   ({
@@ -38,6 +33,22 @@ export const FederatedFolderModal = withLocales(
     const { showAlert } = useAlert()
 
     const [sharingLink, setSharingLink] = useState(null)
+    const {
+      pendingRecipients,
+      setPendingRecipients,
+      selectedOption,
+      setSelectedOption
+    } = usePendingRecipients()
+
+    // share from CozyProvider is wired to callbacks and realtime that
+    // makes existingDocument and existingRecipients reactive to changes.
+    // But we want it to be reactive only when we revoke a member. When we add member
+    // we close the popup and we do not want to see briefly the new contacts added
+    // in members before the modal closes. That's why when clicking on "Share"
+    // we do not use the reactive existingDocument and existingRecipients.
+    const [isSending, setIsSending] = useState(false)
+    const [frozenDoc, setFrozenDoc] = useState(null)
+    const [frozenRecipients, setFrozenRecipients] = useState(null)
 
     const documentPermissions = existingDocument
       ? getDocumentPermissions(existingDocument._id)
@@ -65,32 +76,33 @@ export const FederatedFolderModal = withLocales(
       getSharingById
     ])
 
-    const [federatedRecipients, setFederatedRecipients] = useState({
-      recipients: [],
-      readOnlyRecipients: []
-    })
-    const [folderName] = useState(existingDocument?.name || '')
-
-    const onShare = params => {
-      setFederatedRecipients(prev => ({
-        recipients: mergeAndDeduplicateRecipients([
-          prev.recipients,
-          params.recipients || []
-        ]),
-        readOnlyRecipients: mergeAndDeduplicateRecipients([
-          prev.readOnlyRecipients,
-          params.readOnlyRecipients || []
-        ])
-      }))
-    }
+    const folderName = existingDocument?.name || ''
 
     const onSend = async () => {
+      if (pendingRecipients.length === 0) {
+        onClose()
+        return
+      }
+
+      setIsSending(true)
+      setFrozenDoc(existingDocument)
+      setFrozenRecipients(existingRecipients)
+
       try {
+        const contacts = await getOrCreateFromArray(
+          client,
+          pendingRecipients,
+          contact => client.create('io.cozy.contacts', contact)
+        )
+        const readWriteRecipients =
+          selectedOption === 'readOnly' ? [] : contacts
+        const readOnlyRecipients = selectedOption === 'readOnly' ? contacts : []
+
         await share({
           description: folderName,
           document: existingDocument,
-          recipients: federatedRecipients.recipients,
-          readOnlyRecipients: federatedRecipients.readOnlyRecipients,
+          recipients: readWriteRecipients,
+          readOnlyRecipients,
           sharedDrive: true,
           openSharing: false
         })
@@ -108,31 +120,8 @@ export const FederatedFolderModal = withLocales(
           severity: 'error',
           variant: 'filled'
         })
-      }
-    }
-
-    const onSetType = (index, newType) => {
-      const _id = index.split(RECIPIENT_INDEX_PREFIX)[1]
-
-      if (newType === 'two-way') {
-        setFederatedRecipients(prev => moveRecipientToReadWrite(prev, _id))
-      } else {
-        setFederatedRecipients(prev => moveRecipientToReadOnly(prev, _id))
-      }
-    }
-
-    const onRevoke = async (documentOrIndex, sharingId, memberIndex) => {
-      if (
-        typeof documentOrIndex === 'string' &&
-        documentOrIndex.startsWith(RECIPIENT_INDEX_PREFIX)
-      ) {
-        const _id = documentOrIndex.split(RECIPIENT_INDEX_PREFIX)[1]
-        setFederatedRecipients(prev => ({
-          recipients: prev.recipients.filter(r => r._id !== _id),
-          readOnlyRecipients: prev.readOnlyRecipients.filter(r => r._id !== _id)
-        }))
-      } else {
-        await revoke(documentOrIndex, sharingId, memberIndex)
+      } finally {
+        setIsSending(false)
       }
     }
 
@@ -140,32 +129,26 @@ export const FederatedFolderModal = withLocales(
       ? getRecipients(existingDocument._id)
       : []
 
-    const recipients = [
-      ...existingRecipients,
-      ...formatRecipients(federatedRecipients)
-    ]
-
     const modalTitle = t('FederatedFolder.shareTitle', { name: folderName })
-
     const isSharedDrive = Boolean(existingDocument?.driveId)
 
     return (
       <DumbFederatedFolderModal
         title={modalTitle}
-        document={existingDocument}
-        createContact={contact => client.create('io.cozy.contacts', contact)}
-        recipients={recipients}
-        readOnlyRecipients={federatedRecipients.readOnlyRecipients}
-        currentRecipients={existingRecipients}
-        onRevoke={onRevoke}
-        onSetType={onSetType}
+        document={isSending ? frozenDoc : existingDocument}
+        recipients={isSending ? frozenRecipients : existingRecipients}
+        currentRecipients={isSending ? frozenRecipients : existingRecipients}
+        onRevoke={revoke}
         onSend={onSend}
         onClose={onClose}
-        onShare={onShare}
         sharingLink={sharingLink}
         showShareByEmail={!isSharedDrive}
         autoOpenShareRestriction={autoOpenShareRestriction}
         showGenerateLinkButton={showGenerateLinkButton}
+        pendingRecipients={pendingRecipients}
+        onPendingRecipientsChange={setPendingRecipients}
+        selectedOption={selectedOption}
+        onSelectedOptionChange={setSelectedOption}
       />
     )
   }

package/src/components/FederatedFolder/FederatedFolderModal.spec.jsx

@@ -4,6 +4,8 @@ import React from 'react'
 import { createMockClient } from 'cozy-client'
 
 import { FederatedFolderModal } from './FederatedFolderModal'
+import { getOrCreateFromArray } from '../../helpers/contacts'
+import { usePendingRecipients } from '../../hooks/usePendingRecipients'
 import AppLike from '../SharingBanner/test/AppLike'
 
 const mockShare = jest.fn()
@@ -31,87 +33,45 @@ jest.mock('cozy-ui/transpiled/react/providers/Alert', () => ({
 
 jest.mock('../../hoc/withLocales', () => Component => Component)
 
+jest.mock('../../helpers/contacts', () => ({
+  getOrCreateFromArray: jest.fn()
+}))
+
+jest.mock('../../hooks/usePendingRecipients', () => ({
+  usePendingRecipients: jest.fn()
+}))
+
 jest.mock('./DumbFederatedFolderModal', () => ({
   DumbFederatedFolderModal: ({
     title,
-    createContact,
     recipients,
-    readOnlyRecipients,
     onRevoke,
-    onSetType,
     onSend,
     onClose,
-    onShare,
-    sharingLink
+    sharingLink,
+    pendingRecipients,
+    selectedOption
   }) => (
     <div data-testid="dumb-modal">
       <span data-testid="title">{title}</span>
       <span data-testid="sharing-link">{sharingLink}</span>
       <span data-testid="recipients-count">{recipients.length}</span>
-      <span data-testid="readOnly-recipients-count">
-        {readOnlyRecipients ? readOnlyRecipients.length : 0}
+      <span data-testid="pending-recipients-count">
+        {pendingRecipients.length}
       </span>
-      <button
-        data-testid="btn-share"
-        onClick={() =>
-          onShare({
-            recipients: [{ _id: 'r1', id: 'r1', displayName: 'Alice' }],
-            readOnlyRecipients: [{ _id: 'r2', id: 'r2', displayName: 'Bob' }]
-          })
-        }
-      >
-        Share
-      </button>
-      <button
-        data-testid="btn-share-other"
-        onClick={() =>
-          onShare({
-            recipients: [
-              { _id: 'r1', id: 'r1', displayName: 'Alice' },
-              { _id: 'r3', id: 'r3', displayName: 'Charlie' }
-            ],
-            readOnlyRecipients: []
-          })
-        }
-      >
-        Share Other
-      </button>
+      <span data-testid="selected-option">{selectedOption}</span>
       <button data-testid="btn-send" onClick={onSend}>
         Send
       </button>
-      <button
-        data-testid="btn-set-type-two-way"
-        onClick={() => onSetType('virtual-shared-drive-sharing-r2', 'two-way')}
-      >
-        Set Two-Way
-      </button>
-      <button
-        data-testid="btn-set-type-one-way"
-        onClick={() => onSetType('virtual-shared-drive-sharing-r1', 'one-way')}
-      >
-        Set One-Way
-      </button>
       <button
         data-testid="btn-revoke"
-        onClick={() => onRevoke('virtual-shared-drive-sharing-r1')}
-      >
-        Revoke
-      </button>
-      <button
-        data-testid="btn-revoke-existing"
         onClick={() => onRevoke({ _id: 'folder-123' }, 'abc', 1)}
       >
-        Revoke Existing
+        Revoke
       </button>
       <button data-testid="btn-close" onClick={onClose}>
         Close
       </button>
-      <button
-        data-testid="btn-create-contact"
-        onClick={() => createContact({ email: 'test@example.com' })}
-      >
-        Create Contact
-      </button>
     </div>
   )
 }))
@@ -162,6 +122,13 @@ describe('FederatedFolderModal', () => {
     mockGetRecipients.mockReturnValue([])
     client = createTestClient()
     sharingContextValue = createSharingContextValue()
+    usePendingRecipients.mockReturnValue({
+      pendingRecipients: [],
+      setPendingRecipients: jest.fn(),
+      selectedOption: 'readWrite',
+      setSelectedOption: jest.fn()
+    })
+    getOrCreateFromArray.mockResolvedValue([])
   })
 
   const setup = (props = {}) => {
@@ -211,46 +178,43 @@ describe('FederatedFolderModal', () => {
         expect(getByTestId('sharing-link').textContent).toBe('')
       })
     })
-  })
 
-  describe('onShare callback', () => {
-    it('should update recipients when onShare is called', async () => {
+    it('should pass selectedOption from usePendingRecipients', async () => {
       const { getByTestId } = setup()
 
       await waitFor(() => {
-        expect(getByTestId('recipients-count').textContent).toBe('0')
-      })
-
-      fireEvent.click(getByTestId('btn-share'))
-
-      await waitFor(() => {
-        expect(getByTestId('recipients-count').textContent).toBe('2')
+        expect(getByTestId('selected-option').textContent).toBe('readWrite')
       })
     })
+  })
 
-    it('should accumulate recipients when onShare is called multiple times', async () => {
-      const { getByTestId } = setup()
-
-      await waitFor(() => {
-        expect(getByTestId('recipients-count').textContent).toBe('0')
-      })
+  describe('onSend callback', () => {
+    const pendingRecipient = { name: 'Alice', email: 'alice@example.com' }
 
-      fireEvent.click(getByTestId('btn-share'))
+    it('should call onClose and skip share when there are no pending recipients', async () => {
+      // pendingRecipients defaults to [] via beforeEach
+      const { getByTestId } = setup()
 
       await waitFor(() => {
-        expect(getByTestId('recipients-count').textContent).toBe('2')
+        expect(getByTestId('btn-send')).toBeTruthy()
       })
 
-      fireEvent.click(getByTestId('btn-share-other'))
+      fireEvent.click(getByTestId('btn-send'))
 
       await waitFor(() => {
-        expect(getByTestId('recipients-count').textContent).toBe('3')
+        expect(mockOnClose).toHaveBeenCalled()
+        expect(mockShare).not.toHaveBeenCalled()
       })
     })
-  })
 
-  describe('onSend callback', () => {
     it('should call share with correct parameters', async () => {
+      usePendingRecipients.mockReturnValue({
+        pendingRecipients: [pendingRecipient],
+        setPendingRecipients: jest.fn(),
+        selectedOption: 'readWrite',
+        setSelectedOption: jest.fn()
+      })
+      getOrCreateFromArray.mockResolvedValue([pendingRecipient])
       mockShare.mockResolvedValueOnce({})
       const { getByTestId } = setup()
 
@@ -258,15 +222,14 @@ describe('FederatedFolderModal', () => {
         expect(getByTestId('btn-send')).toBeTruthy()
       })
 
-      fireEvent.click(getByTestId('btn-share'))
       fireEvent.click(getByTestId('btn-send'))
 
       await waitFor(() => {
         expect(mockShare).toHaveBeenCalledWith({
           description: 'My Test Folder',
           document: mockDocument,
-          recipients: [{ _id: 'r1', id: 'r1', displayName: 'Alice' }],
-          readOnlyRecipients: [{ _id: 'r2', id: 'r2', displayName: 'Bob' }],
+          recipients: [pendingRecipient],
+          readOnlyRecipients: [],
           sharedDrive: true,
           openSharing: false
         })
@@ -274,6 +237,13 @@ describe('FederatedFolderModal', () => {
     })
 
     it('should show success alert and close modal on successful share', async () => {
+      usePendingRecipients.mockReturnValue({
+        pendingRecipients: [pendingRecipient],
+        setPendingRecipients: jest.fn(),
+        selectedOption: 'readWrite',
+        setSelectedOption: jest.fn()
+      })
+      getOrCreateFromArray.mockResolvedValue([pendingRecipient])
       mockShare.mockResolvedValueOnce({})
       const { getByTestId } = setup()
 
@@ -294,6 +264,13 @@ describe('FederatedFolderModal', () => {
     })
 
     it('should show error alert when share fails', async () => {
+      usePendingRecipients.mockReturnValue({
+        pendingRecipients: [pendingRecipient],
+        setPendingRecipients: jest.fn(),
+        selectedOption: 'readWrite',
+        setSelectedOption: jest.fn()
+      })
+      getOrCreateFromArray.mockResolvedValue([pendingRecipient])
       mockShare.mockRejectedValueOnce(new Error('Share failed'))
       const { getByTestId } = setup()
 
@@ -314,75 +291,19 @@ describe('FederatedFolderModal', () => {
     })
   })
 
-  describe('onSetType callback', () => {
-    it('should move recipient from readOnlyRecipients to recipients when setting two-way', async () => {
-      const { getByTestId } = setup()
-
-      await waitFor(() => {
-        expect(getByTestId('btn-share')).toBeTruthy()
-      })
-
-      fireEvent.click(getByTestId('btn-share'))
-
-      // After btn-share: combined recipients = 2 (1 readWrite + 1 readOnly)
-      await waitFor(() => {
-        expect(getByTestId('recipients-count').textContent).toBe('2')
-        expect(getByTestId('readOnly-recipients-count').textContent).toBe('1')
-      })
-
-      fireEvent.click(getByTestId('btn-set-type-two-way'))
-
-      // After setting two-way: Bob moves from readOnlyRecipients to recipients
-      await waitFor(() => {
-        expect(getByTestId('recipients-count').textContent).toBe('2')
-        expect(getByTestId('readOnly-recipients-count').textContent).toBe('0')
-      })
-    })
-
-    it('should move recipient from recipients to readOnlyRecipients when setting one-way', async () => {
-      const { getByTestId } = setup()
-
-      await waitFor(() => {
-        expect(getByTestId('btn-share')).toBeTruthy()
-      })
-
-      fireEvent.click(getByTestId('btn-share'))
-
-      // After btn-share: combined recipients = 2 (1 readWrite + 1 readOnly)
-      await waitFor(() => {
-        expect(getByTestId('recipients-count').textContent).toBe('2')
-        expect(getByTestId('readOnly-recipients-count').textContent).toBe('1')
-      })
-
-      fireEvent.click(getByTestId('btn-set-type-one-way'))
-
-      // After setting one-way: Alice moves from recipients to readOnlyRecipients
-      // Combined count stays at 2, but readOnlyRecipients increases from 1 to 2
-      await waitFor(() => {
-        expect(getByTestId('recipients-count').textContent).toBe('2')
-        expect(getByTestId('readOnly-recipients-count').textContent).toBe('2')
-      })
-    })
-  })
-
   describe('onRevoke callback', () => {
-    it('should remove recipient from both lists', async () => {
+    it('should call revoke from sharing context for existing members', async () => {
+      mockRevoke.mockResolvedValueOnce({})
       const { getByTestId } = setup()
 
       await waitFor(() => {
-        expect(getByTestId('btn-share')).toBeTruthy()
-      })
-
-      fireEvent.click(getByTestId('btn-share'))
-
-      await waitFor(() => {
-        expect(getByTestId('recipients-count').textContent).toBe('2')
+        expect(getByTestId('dumb-modal')).toBeTruthy()
       })
 
       fireEvent.click(getByTestId('btn-revoke'))
 
       await waitFor(() => {
-        expect(getByTestId('recipients-count').textContent).toBe('1')
+        expect(mockRevoke).toHaveBeenCalledWith({ _id: 'folder-123' }, 'abc', 1)
       })
     })
   })
@@ -431,49 +352,5 @@ describe('FederatedFolderModal', () => {
         expect(getByTestId('recipients-count').textContent).toBe('2')
       })
     })
-
-    it('should pass existing recipients as currentRecipients', async () => {
-      mockGetRecipients.mockReturnValue(existingRecipients)
-      const { getByTestId } = setup()
-
-      await waitFor(() => {
-        expect(getByTestId('recipients-count').textContent).toBe('2')
-      })
-    })
-
-    it('should call revoke for existing member', async () => {
-      mockGetRecipients.mockReturnValue(existingRecipients)
-      mockRevoke.mockResolvedValueOnce({})
-      const { getByTestId } = setup()
-
-      await waitFor(() => {
-        expect(getByTestId('dumb-modal')).toBeTruthy()
-      })
-
-      // Simulate revoke of an existing member (non-virtual-shared-drive index)
-      fireEvent.click(getByTestId('btn-revoke-existing'))
-
-      await waitFor(() => {
-        expect(mockRevoke).toHaveBeenCalledWith({ _id: 'folder-123' }, 'abc', 1)
-      })
-    })
-  })
-
-  describe('createContact callback', () => {
-    it('should call client.create when creating a contact', async () => {
-      const { getByTestId } = setup()
-
-      await waitFor(() => {
-        expect(getByTestId('btn-create-contact')).toBeTruthy()
-      })
-
-      fireEvent.click(getByTestId('btn-create-contact'))
-
-      await waitFor(() => {
-        expect(client.create).toHaveBeenCalledWith('io.cozy.contacts', {
-          email: 'test@example.com'
-        })
-      })
-    })
   })
 })

package/src/components/Recipient/GroupRecipientPermissions.jsx

@@ -1,20 +1,18 @@
 import React, { useState, useRef } from 'react'
 
-import { useClient } from 'cozy-client'
 import ActionsMenu from 'cozy-ui/transpiled/react/ActionsMenu'
-import {
-  makeActions,
-  divider
-} from 'cozy-ui/transpiled/react/ActionsMenu/Actions'
+import { makeActions } from 'cozy-ui/transpiled/react/ActionsMenu/Actions'
 import ActionsMenuMobileHeader from 'cozy-ui/transpiled/react/ActionsMenu/ActionsMenuMobileHeader'
 import DropdownButton from 'cozy-ui/transpiled/react/DropdownButton'
+import Icon from 'cozy-ui/transpiled/react/Icon'
+import IconButton from 'cozy-ui/transpiled/react/IconButton'
+import CrossCircleOutlineIcon from 'cozy-ui/transpiled/react/Icons/CrossCircleOutline'
 import ListItemIcon from 'cozy-ui/transpiled/react/ListItemIcon'
 import ListItemText from 'cozy-ui/transpiled/react/ListItemText'
 import Spinner from 'cozy-ui/transpiled/react/Spinner'
 import { useI18n } from 'twake-i18n'
 
 import { permission } from './actions/permission'
-import { revokeGroup as revokeGroupAction } from './actions/revokeGroup'
 import { useSharingContext } from '../../hooks/useSharingContext'
 import { GroupAvatar } from '../Avatar/GroupAvatar'
 
@@ -30,7 +28,6 @@ const GroupRecipientPermissions = ({
 }) => {
   const { t } = useI18n()
   const buttonRef = useRef()
-  const client = useClient()
   const { revokeGroup, revokeSelf } = useSharingContext()
 
   const [isMenuDisplayed, setMenuDisplayed] = useState(false)
@@ -53,12 +50,9 @@ const GroupRecipientPermissions = ({
 
   const type = isReadOnly ? 'one-way' : 'two-way'
 
-  const actions = makeActions([permission, divider, revokeGroupAction], {
-    client,
+  const actions = makeActions([permission], {
     t,
-    type,
-    isOwner,
-    handleRevocation
+    type
   })
 
   return (
@@ -73,7 +67,7 @@ const GroupRecipientPermissions = ({
             onClick={toggleMenu}
             textVariant="body2"
           >
-            {t(`Share.type.${type}`).toLowerCase()}
+            {t(`Share.type.${type}`)}
           </DropdownButton>
           <ActionsMenu
             ref={buttonRef}
@@ -92,6 +86,14 @@ const GroupRecipientPermissions = ({
               />
             </ActionsMenuMobileHeader>
           </ActionsMenu>
+          <IconButton
+            onClick={handleRevocation}
+            size="small"
+            className="u-ml-half"
+            aria-label={t('Share.members.revoke')}
+          >
+            <Icon icon={CrossCircleOutlineIcon} />
+          </IconButton>
         </>
       )}
     </div>

package/src/components/Recipient/LinkRecipient.jsx

@@ -2,14 +2,15 @@ import PropTypes from 'prop-types'
 import React, { useState } from 'react'
 
 import Avatar from 'cozy-ui/transpiled/react/Avatar'
-import Button from 'cozy-ui/transpiled/react/Button'
 import Fade from 'cozy-ui/transpiled/react/Fade'
 import Icon from 'cozy-ui/transpiled/react/Icon'
+import IconButton from 'cozy-ui/transpiled/react/IconButton'
 import Gear from 'cozy-ui/transpiled/react/Icons/Gear'
 import LinkIcon from 'cozy-ui/transpiled/react/Icons/Link'
 import ListItem from 'cozy-ui/transpiled/react/ListItem'
 import ListItemIcon from 'cozy-ui/transpiled/react/ListItemIcon'
 import ListItemText from 'cozy-ui/transpiled/react/ListItemText'
+import Typography from 'cozy-ui/transpiled/react/Typography'
 import useBreakpoints from 'cozy-ui/transpiled/react/providers/Breakpoints'
 import { useI18n } from 'twake-i18n'
 
@@ -68,15 +69,14 @@ const LinkRecipient = props => {
           </Avatar>
         </ListItemIcon>
         <ListItemText
+          onClick={() => setOpenShareRestrictionModal(true)}
           primary={
-            <Button
-              variant="ghost"
-              onClick={() => setOpenShareRestrictionModal(true)}
-              endIcon={<Icon icon={Gear} />}
-              className="u-p-0"
-            >
-              {textPrimary}
-            </Button>
+            <div className="u-flex u-flex-items-center">
+              <Typography className="u-c-pointer ">{textPrimary}</Typography>
+              <IconButton size="small" className="u-ml-half">
+                <Icon icon={Gear} />
+              </IconButton>
+            </div>
           }
           secondary={textSecondary}
         />

package/src/components/Recipient/LinkRecipientLite.jsx

@@ -32,9 +32,7 @@ const LinkRecipientLite = ({ permissions, link }) => {
         secondary={link}
       />
       <Typography className="u-flex-shrink-0" variant="body2">
-        {t(
-          `Share.type.${isReadOnlyPermissions ? 'one-way' : 'two-way'}`
-        ).toLowerCase()}
+        {t(`Share.type.${isReadOnlyPermissions ? 'one-way' : 'two-way'}`)}
       </Typography>
     </ListItem>
   )

package/src/components/Recipient/LinkRecipientPermissions.jsx

@@ -38,6 +38,7 @@ const LinkRecipientPermissions = ({ className, document }) => {
   const handleSetType = newType => {
     const verbs = newType === 'one-way' ? READ_ONLY_PERMS : WRITE_PERMS
     updateDocumentPermissions(document, { verbs })
+    hideMenu()
   }
 
   const handleRevocation = () => revokeSharingLink(document)
@@ -63,7 +64,7 @@ const LinkRecipientPermissions = ({ className, document }) => {
           textVariant="body2"
           onClick={toggleMenu}
         >
-          {t(`Share.type.${type}`).toLowerCase()}
+          {t(`Share.type.${type}`)}
         </DropdownButton>
         <ActionsMenu
           ref={buttonRef}