cozy-sharing 37.5.4 → 37.6.0

package/src/SharingProvider.spec.jsx

@@ -533,7 +533,8 @@ describe('shared drive recipient revocation', () => {
     sharingCol = {
       addRecipients: jest.fn(),
       create: jest.fn().mockResolvedValue({ data: NEW_SHARING }),
-      revokeRecipient: jest.fn().mockResolvedValue({})
+      revokeRecipient: jest.fn().mockResolvedValue({}),
+      fetchEffectiveRecipients: jest.fn().mockResolvedValue({ data: [] })
     }
     provider.sharingCol = sharingCol
   })
@@ -792,7 +793,8 @@ describe('updateSharingMemberType', () => {
     )
     instance.sharingCol = {
       setReadOnly: jest.fn().mockResolvedValue({}),
-      setReadWrite: jest.fn().mockResolvedValue({})
+      setReadWrite: jest.fn().mockResolvedValue({}),
+      fetchEffectiveRecipients: jest.fn().mockResolvedValue({ data: [] })
     }
   })
 
@@ -913,6 +915,12 @@ describe('updateSharingMemberType', () => {
       }
     })
     expect(instance.dispatch).toHaveBeenNthCalledWith(2, {
+      type: 'UPDATE_EFFECTIVE_RECIPIENT',
+      sharingId: 'sharing-123',
+      memberIndex: 1,
+      readOnly: true
+    })
+    expect(instance.dispatch).toHaveBeenNthCalledWith(3, {
       type: 'UPDATE_SHARING',
       sharing: {
         ...mockSharing,
@@ -928,6 +936,12 @@ describe('updateSharingMemberType', () => {
         }
       }
     })
+    expect(instance.dispatch).toHaveBeenNthCalledWith(4, {
+      type: 'UPDATE_EFFECTIVE_RECIPIENT',
+      sharingId: 'sharing-123',
+      memberIndex: 1,
+      readOnly: false
+    })
   })
 
   it('should preserve realtime changes when rolling back an API failure', async () => {
@@ -964,6 +978,12 @@ describe('updateSharingMemberType', () => {
     ).rejects.toThrow('Network error')
 
     expect(instance.dispatch).toHaveBeenNthCalledWith(2, {
+      type: 'UPDATE_EFFECTIVE_RECIPIENT',
+      sharingId: 'sharing-123',
+      memberIndex: 1,
+      readOnly: true
+    })
+    expect(instance.dispatch).toHaveBeenNthCalledWith(3, {
       type: 'UPDATE_SHARING',
       sharing: {
         ...realtimeSharing,
@@ -979,6 +999,12 @@ describe('updateSharingMemberType', () => {
         }
       }
     })
+    expect(instance.dispatch).toHaveBeenNthCalledWith(4, {
+      type: 'UPDATE_EFFECTIVE_RECIPIENT',
+      sharingId: 'sharing-123',
+      memberIndex: 1,
+      readOnly: false
+    })
   })
 })
 

package/src/components/FederatedFolder/FederatedFolderModal.jsx

@@ -16,7 +16,6 @@ import { getOrCreateFromArray } from '../../helpers/contacts'
 import withLocales from '../../hoc/withLocales'
 import { usePendingRecipients } from '../../hooks/usePendingRecipients'
 import { useSharingContext } from '../../hooks/useSharingContext'
-import { getRecipientsFromSharing } from '../../state'
 import styles from '../../styles/share.styl'
 import AntivirusAlert from '../AntivirusAlert'
 import { default as DumbShareByEmail } from '../ShareByEmail'
@@ -29,6 +28,7 @@ const FederatedFolderModalContent = ({
   onClose,
   onRevokeSuccess,
   document: existingDocument,
+  recipients,
   autoOpenShareRestriction,
   showGenerateLinkButton
 }) => {
@@ -48,10 +48,6 @@ const FederatedFolderModalContent = ({
     getFederatedShareLink,
     getDocumentPermissions,
     fetchSharedDriveSharingLinks,
-    getRecipients,
-    getSharedParentPath,
-    hasSharedChild,
-    hasSharedParent,
     isOwner,
     revoke,
     revokeSelf
@@ -70,11 +66,9 @@ const FederatedFolderModalContent = ({
     useState(null)
   const [frozenDoc, setFrozenDoc] = useState(null)
   const [frozenRecipients, setFrozenRecipients] = useState(null)
-  const [sharedParentFolder, setSharedParentFolder] = useState(null)
 
   const documentId = existingDocument?._id || existingDocument?.id
   const folderName = existingDocument?.name || ''
-  const documentPath = existingDocument?.path
   const sharedDriveSharing = existingDocument?.driveId
     ? getSharingById(existingDocument.driveId)
     : null
@@ -83,25 +77,7 @@ const FederatedFolderModalContent = ({
       (ids, rule) => ids.concat(rule.values || []),
       []
     ) || []
-  // Owner case: when document is inside a shared folder (via path) but has no driveId,
-  // find the parent shared folder to use its _id for recipient/permission lookups.
-  const hasSharedParentByPath = Boolean(
-    documentPath && hasSharedParent(documentPath)
-  )
-  const sharedParentPath =
-    !sharedDriveSharing && hasSharedParentByPath && documentPath
-      ? getSharedParentPath(documentPath)
-      : null
-  const sharingReferenceId =
-    !sharedDriveSharing && sharedParentFolder?._id
-      ? sharedParentFolder._id
-      : documentId
-  const existingRecipients =
-    sharedDriveSharing?.attributes?.members && documentId
-      ? getRecipientsFromSharing(sharedDriveSharing, documentId)
-      : sharingReferenceId
-        ? getRecipients(sharingReferenceId)
-        : []
+
   const documentPermissions = documentId
     ? getDocumentPermissions(documentId)
     : []
@@ -124,11 +100,6 @@ const FederatedFolderModalContent = ({
       ? isOwner(documentId)
       : false
   const isMemberReadOnly = isInsideSharedDrive && !isCurrentUserOwner
-  const hasParentRestriction = isInsideSharedDrive || hasSharedParentByPath
-  const hasChildRestriction = Boolean(
-    documentPath && hasSharedChild(documentPath)
-  )
-  const canShareByEmail = !hasParentRestriction && !hasChildRestriction
   const canManageSharing =
     isCurrentUserOwner || (documentId ? canReshare(documentId) : false)
 
@@ -181,7 +152,7 @@ const FederatedFolderModalContent = ({
 
     setIsSending(true)
     setFrozenDoc(existingDocument)
-    setFrozenRecipients(existingRecipients)
+    setFrozenRecipients(recipients)
 
     try {
       const contacts = await getOrCreateFromArray(
@@ -219,21 +190,6 @@ const FederatedFolderModalContent = ({
     }
   }
 
-  useEffect(() => {
-    if (!sharedParentPath) return
-    const fetchParentFolder = async () => {
-      try {
-        const res = await client
-          .collection('io.cozy.files')
-          .statByPath(sharedParentPath)
-        if (res.data) setSharedParentFolder(res.data)
-      } catch (err) {
-        log.error('Failed to fetch shared parent folder', err)
-      }
-    }
-    fetchParentFolder()
-  }, [sharedParentPath, client])
-
   useEffect(() => {
     if (!existingDocument?.driveId) return
     if (!documentId) return
@@ -283,7 +239,7 @@ const FederatedFolderModalContent = ({
             <AntivirusAlert
               document={isSending ? frozenDoc : existingDocument}
             />
-            {!canShareByEmail && (
+            {isInsideSharedDrive ? (
               <div className={styles['share-byemail-onlybylink']}>
                 {t('Files.share.shareByEmail.onlyByLink', {
                   type: t(
@@ -292,26 +248,15 @@ const FederatedFolderModalContent = ({
                     }`
                   )
                 })}{' '}
-                <strong>
-                  {t(
-                    `Files.share.shareByEmail.${
-                      hasParentRestriction
-                        ? 'hasSharedParent'
-                        : 'hasSharedChild'
-                    }`
-                  )}
-                </strong>
+                <strong>{t('Files.share.shareByEmail.hasSharedParent')}</strong>
               </div>
-            )}
-            {canShareByEmail && (
+            ) : (
               <>
                 <Typography variant="h6" className="u-mt-1-half u-mb-half">
                   {t('Share.contacts.addUsers')}
                 </Typography>
                 <DumbShareByEmail
-                  currentRecipients={
-                    isSending ? frozenRecipients : existingRecipients
-                  }
+                  currentRecipients={isSending ? frozenRecipients : recipients}
                   document={isSending ? frozenDoc : existingDocument}
                   documentType="Files"
                   pendingRecipients={pendingRecipients}
@@ -329,7 +274,7 @@ const FederatedFolderModalContent = ({
             isSharedDrive
             canManageMembers={!isMemberReadOnly}
             canManageLink={true}
-            recipients={isSending ? frozenRecipients : existingRecipients}
+            recipients={isSending ? frozenRecipients : recipients}
             document={isSending ? frozenDoc : existingDocument}
             documentType="Files"
             className={styles['share-dialog-members']}
@@ -348,7 +293,7 @@ const FederatedFolderModalContent = ({
             showGenerateLinkButton={showGenerateLinkButton}
             autoOpenShareRestriction={autoOpenShareRestriction}
           />
-          {canShareByEmail && (
+          {!isInsideSharedDrive && (
             <Button
               variant="primary"
               label={t('FederatedFolder.share')}
@@ -362,9 +307,49 @@ const FederatedFolderModalContent = ({
   )
 }
 
+FederatedFolderModalContent.propTypes = {
+  onClose: PropTypes.func.isRequired,
+  onRevokeSuccess: PropTypes.func,
+  document: PropTypes.object.isRequired,
+  recipients: PropTypes.array,
+  autoOpenShareRestriction: PropTypes.bool,
+  showGenerateLinkButton: PropTypes.bool
+}
+
+const FederatedFolderModalWrapper = ({ document, ...props }) => {
+  const fileId = document._id || document.id
+  const {
+    getEffectiveRecipients,
+    fetchEffectiveRecipients,
+    invalidateEffectiveRecipients
+  } = useSharingContext()
+
+  useEffect(() => {
+    fetchEffectiveRecipients(fileId, document.driveId).catch(log.error)
+    return () => {
+      invalidateEffectiveRecipients(fileId)
+    }
+  }, [
+    fileId,
+    document.driveId,
+    fetchEffectiveRecipients,
+    invalidateEffectiveRecipients
+  ])
+
+  const recipients = getEffectiveRecipients(fileId)
+
+  return (
+    <FederatedFolderModalContent
+      document={document}
+      recipients={recipients}
+      {...props}
+    />
+  )
+}
+
 export const FederatedFolderModal = withLocales(props => (
   <ConfirmDialogProvider>
-    <FederatedFolderModalContent {...props} />
+    <FederatedFolderModalWrapper {...props} />
   </ConfirmDialogProvider>
 ))
 

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

@@ -16,7 +16,6 @@ const mockIsOwner = jest.fn()
 const mockCanReshare = jest.fn()
 const mockGetSharingLink = jest.fn()
 const mockGetFederatedShareLink = jest.fn()
-const mockGetRecipients = jest.fn().mockReturnValue([])
 const mockGetSharingById = jest.fn()
 const mockHasSharedChild = jest.fn()
 const mockHasSharedParent = jest.fn()
@@ -25,6 +24,8 @@ const mockGetDocumentPermissions = jest.fn().mockReturnValue([])
 const mockShowAlert = jest.fn()
 const mockOnClose = jest.fn()
 
+const mockGetEffectiveRecipients = jest.fn(() => [])
+
 jest.mock('../../hooks/useSharingContext', () => ({
   useSharingContext: () => ({
     share: mockShare,
@@ -38,12 +39,14 @@ jest.mock('../../hooks/useSharingContext', () => ({
     getDocumentPermissions: mockGetDocumentPermissions,
     fetchSharedDriveSharingLinks: mockFetchSharedDriveSharingLinks,
     getOwner: jest.fn(),
-    getRecipients: mockGetRecipients,
     getSharedParentPath: jest.fn().mockReturnValue(null),
     getSharingById: mockGetSharingById,
     hasSharedChild: mockHasSharedChild,
     hasSharedParent: mockHasSharedParent,
-    isOrgSharedDrive: jest.fn(() => false)
+    isOrgSharedDrive: jest.fn(() => false),
+    getEffectiveRecipients: (...args) => mockGetEffectiveRecipients(...args),
+    fetchEffectiveRecipients: jest.fn().mockResolvedValue(),
+    invalidateEffectiveRecipients: jest.fn()
   })
 }))
 
@@ -118,7 +121,6 @@ describe('FederatedFolderModal', () => {
     mockGetFederatedShareLink.mockReturnValue(
       'https://example.com/share/federated-abc123'
     )
-    mockGetRecipients.mockReturnValue([])
     mockGetSharingById.mockReturnValue(null)
     mockHasSharedChild.mockReturnValue(false)
     mockHasSharedParent.mockReturnValue(false)
@@ -458,12 +460,15 @@ describe('FederatedFolderModal', () => {
       instance: 'https://me.mycozy.cloud',
       memberIndex: 1,
       sharingId: 'sharing-id',
-      type: 'two-way'
+      type: 'two-way',
+      status: 'ready',
+      name: 'Me',
+      email: 'me@test.com'
     }
 
     it('should call onRevokeSelf and onRevokeSuccess when the current user is revoked', async () => {
       const onRevokeSuccess = jest.fn()
-      mockGetRecipients.mockReturnValue([currentUserRecipient])
+      mockGetEffectiveRecipients.mockReturnValue([currentUserRecipient])
 
       const { findByLabelText } = setup({ onRevokeSuccess })
 
@@ -481,10 +486,13 @@ describe('FederatedFolderModal', () => {
         instance: 'https://other.mycozy.cloud',
         memberIndex: 2,
         sharingId: 'sharing-id',
-        type: 'two-way'
+        type: 'two-way',
+        status: 'ready',
+        name: 'Other',
+        email: 'other@test.com'
       }
       mockCanReshare.mockReturnValue(true)
-      mockGetRecipients.mockReturnValue([otherRecipient])
+      mockGetEffectiveRecipients.mockReturnValue([otherRecipient])
 
       const { findByLabelText } = setup()
 

package/src/components/ShareDialogCozyToCozy.jsx

@@ -1,106 +1,20 @@
-import cx from 'classnames'
 import PropTypes from 'prop-types'
 import React, { useCallback, useMemo, useState } from 'react'
 
 import { useClient } from 'cozy-client'
 import Button from 'cozy-ui/transpiled/react/Buttons'
-import Typography from 'cozy-ui/transpiled/react/Typography'
 import { useAlert } from 'cozy-ui/transpiled/react/providers/Alert'
-import useBreakpoints from 'cozy-ui/transpiled/react/providers/Breakpoints'
 import { useI18n } from 'twake-i18n'
 
-import AntivirusAlert from './AntivirusAlert'
-import { default as DumbShareByEmail } from './ShareByEmail'
 import { default as DumbShareByLink } from './ShareByLink'
 import ShareDialogTwoStepsConfirmationContainer from './ShareDialogTwoStepsConfirmationContainer'
 import { ShareRecipientsLimitModal } from './ShareRecipientsLimitModal'
-import WhoHasAccess from './WhoHasAccess'
+import SharingContent from './SharingContent'
 import { getOrCreateFromArray } from '../helpers/contacts'
 import { hasReachRecipientsLimit } from '../helpers/recipients'
 import { getErrorMessage, getSuccessMessage } from '../helpers/share'
 import { usePendingRecipients } from '../hooks/usePendingRecipients'
 import { useSharingContext } from '../hooks/useSharingContext'
-import styles from '../styles/share.styl'
-
-const SharingContent = ({
-  document,
-  documentType,
-  hasSharedParent,
-  isOwner,
-  onRevoke,
-  onRevokeSelf,
-  recipients,
-  sharing,
-  showShareByEmail,
-  showShareOnlyByLink,
-  showWhoHasAccess,
-  recipientsToBeConfirmed,
-  verifyRecipient,
-  link,
-  permissions,
-  pendingRecipients,
-  onPendingRecipientsChange,
-  selectedOption,
-  onSelectedOptionChange
-}) => {
-  const { t } = useI18n()
-  const { isMobile } = useBreakpoints()
-  return (
-    <div className={cx(styles['share-modal-content'])}>
-      <div className={cx('u-pt-1-half', isMobile ? 'u-ph-1' : 'u-ph-2')}>
-        <AntivirusAlert document={document} />
-        {showShareOnlyByLink && (
-          <div className={styles['share-byemail-onlybylink']}>
-            {t(`${documentType}.share.shareByEmail.onlyByLink`, {
-              type: t(
-                `${documentType}.share.shareByEmail.type.${
-                  document.type === 'directory' ? 'folder' : 'file'
-                }`
-              )
-            })}{' '}
-            <strong>
-              {t(
-                `${documentType}.share.shareByEmail.${
-                  hasSharedParent ? 'hasSharedParent' : 'hasSharedChild'
-                }`
-              )}
-            </strong>
-          </div>
-        )}
-        <Typography variant="h6" className="u-mb-half">
-          {t('Share.contacts.addUsers')}
-        </Typography>
-        {showShareByEmail && (
-          <DumbShareByEmail
-            currentRecipients={recipients}
-            document={document}
-            documentType={documentType}
-            sharing={sharing}
-            pendingRecipients={pendingRecipients}
-            onPendingRecipientsChange={onPendingRecipientsChange}
-            selectedOption={selectedOption}
-            onSelectedOptionChange={onSelectedOptionChange}
-            enableCreateContact
-          />
-        )}
-      </div>
-      {showWhoHasAccess && (
-        <WhoHasAccess
-          document={document}
-          documentType={documentType}
-          isOwner={isOwner}
-          onRevoke={onRevoke}
-          onRevokeSelf={onRevokeSelf}
-          recipients={recipients}
-          recipientsToBeConfirmed={recipientsToBeConfirmed}
-          verifyRecipient={verifyRecipient}
-          link={link}
-          permissions={permissions}
-        />
-      )}
-    </div>
-  )
-}
 
 const SharingTitleFunction = ({ documentType, document }) =>
   function SharingTitle() {

package/src/components/ShareModal/EditableSharingModal.jsx

@@ -1,21 +1,23 @@
 import PropTypes from 'prop-types'
-import React from 'react'
+import React, { useEffect } from 'react'
 
 import { useClient } from 'cozy-client'
+import flag from 'cozy-flags'
+import minilog from 'cozy-minilog'
 
 import { useFetchDocumentPath } from '../../hooks/useFetchDocumentPath'
 import { useSharingContext } from '../../hooks/useSharingContext'
 import { Contact } from '../../models'
 import { ShareModal } from '../ShareModal'
 
-export const EditableSharingModal = ({ document, ...rest }) => {
+export const EditableSharingModal = ({ document, recipients, ...rest }) => {
   const client = useClient()
+  const isFederatedMode = flag('drive.federated-shared-folder.enabled')
   const documentPath = useFetchDocumentPath(client, document)
 
   const {
     documentType,
     getDocumentPermissions,
-    getRecipients,
     getSharingForSelf,
     getSharingLink,
     hasSharedChild,
@@ -31,15 +33,23 @@ export const EditableSharingModal = ({ document, ...rest }) => {
       createContact={contact => client.create(Contact.doctype, contact)}
       document={document}
       documentType={documentType}
-      hasSharedChild={documentPath && hasSharedChild(documentPath)}
-      hasSharedParent={documentPath && hasSharedParent(documentPath)}
+      hasSharedChild={
+        isFederatedMode
+          ? undefined
+          : documentPath && hasSharedChild(documentPath)
+      }
+      hasSharedParent={
+        isFederatedMode
+          ? undefined
+          : documentPath && hasSharedParent(documentPath)
+      }
       isOwner={isOwner(document._id)}
       link={getSharingLink(document._id)}
       onRevoke={revoke}
       onRevokeSelf={revokeSelf}
       onShare={share}
       permissions={getDocumentPermissions(document._id)}
-      recipients={getRecipients(document._id)}
+      recipients={recipients}
       sharing={getSharingForSelf(document._id)}
       {...rest}
     />
@@ -47,7 +57,50 @@ export const EditableSharingModal = ({ document, ...rest }) => {
 }
 
 EditableSharingModal.propTypes = {
-  document: PropTypes.object
+  document: PropTypes.object,
+  recipients: PropTypes.array
+}
+
+export const EditableSharingModalWrapper = ({ document, ...rest }) => {
+  const log = minilog('EditableSharingModal')
+  const isFederatedMode = flag('drive.federated-shared-folder.enabled')
+  const {
+    getRecipients,
+    getEffectiveRecipients,
+    fetchEffectiveRecipients,
+    invalidateEffectiveRecipients
+  } = useSharingContext()
+
+  const fileId = document._id || document.id
+
+  useEffect(() => {
+    if (isFederatedMode && fileId) {
+      fetchEffectiveRecipients(fileId, document.driveId).catch(log.error)
+    }
+    return () => {
+      if (fileId) {
+        invalidateEffectiveRecipients(fileId)
+      }
+    }
+  }, [
+    fileId,
+    document.driveId,
+    isFederatedMode,
+    fetchEffectiveRecipients,
+    invalidateEffectiveRecipients
+  ])
+
+  const recipients = isFederatedMode
+    ? getEffectiveRecipients(fileId)
+    : getRecipients(fileId)
+
+  return (
+    <EditableSharingModal
+      document={document}
+      recipients={recipients}
+      {...rest}
+    />
+  )
 }
 
-export default EditableSharingModal
+export default EditableSharingModalWrapper

package/src/components/ShareModal/ShareModal.jsx

@@ -6,10 +6,10 @@ import { useLocation } from 'react-router'
 import flag from 'cozy-flags'
 
 import EditableSharingModal from './EditableSharingModal'
-import { SharingDetailsModal } from './SharingDetailsModal'
+import SharingDetailsModal from './SharingDetailsModal'
 import withLocales from '../../hoc/withLocales'
 import { useSharingContext } from '../../hooks/useSharingContext'
-import { FederatedFolderModal } from '../FederatedFolder/FederatedFolderModal'
+import FederatedFolderModal from '../FederatedFolder/FederatedFolderModal'
 
 export const ShareModal = withLocales(props => {
   const location = useLocation?.()

package/src/components/ShareModal.jsx

@@ -75,8 +75,6 @@ export const ShareModal = ({
   )
 }
 
-export default ShareModal
-
 ShareModal.propTypes = {
   autoOpenShareRestriction: PropTypes.bool,
   createContact: PropTypes.func.isRequired,
@@ -100,3 +98,5 @@ ShareModal.propTypes = {
     recipientConfirmationDialogContent: PropTypes.func
   })
 }
+
+export default ShareModal

package/src/components/SharingContent.jsx

@@ -0,0 +1,94 @@
+import cx from 'classnames'
+import React from 'react'
+
+import Typography from 'cozy-ui/transpiled/react/Typography'
+import useBreakpoints from 'cozy-ui/transpiled/react/providers/Breakpoints'
+import { useI18n } from 'twake-i18n'
+
+import AntivirusAlert from './AntivirusAlert'
+import { default as DumbShareByEmail } from './ShareByEmail'
+import WhoHasAccess from './WhoHasAccess'
+import styles from '../styles/share.styl'
+
+const SharingContent = ({
+  document,
+  documentType,
+  hasSharedParent,
+  isOwner,
+  onRevoke,
+  onRevokeSelf,
+  recipients,
+  sharing,
+  showShareByEmail,
+  showShareOnlyByLink,
+  showWhoHasAccess,
+  recipientsToBeConfirmed,
+  verifyRecipient,
+  link,
+  permissions,
+  pendingRecipients,
+  onPendingRecipientsChange,
+  selectedOption,
+  onSelectedOptionChange
+}) => {
+  const { t } = useI18n()
+  const { isMobile } = useBreakpoints()
+
+  return (
+    <div className={cx(styles['share-modal-content'])}>
+      <div className={cx('u-pt-1-half', isMobile ? 'u-ph-1' : 'u-ph-2')}>
+        <AntivirusAlert document={document} />
+        {showShareOnlyByLink && (
+          <div className={styles['share-byemail-onlybylink']}>
+            {t(`${documentType}.share.shareByEmail.onlyByLink`, {
+              type: t(
+                `${documentType}.share.shareByEmail.type.${
+                  document.type === 'directory' ? 'folder' : 'file'
+                }`
+              )
+            })}{' '}
+            <strong>
+              {t(
+                `${documentType}.share.shareByEmail.${
+                  hasSharedParent ? 'hasSharedParent' : 'hasSharedChild'
+                }`
+              )}
+            </strong>
+          </div>
+        )}
+        <Typography variant="h6" className="u-mb-half">
+          {t('Share.contacts.addUsers')}
+        </Typography>
+        {showShareByEmail && (
+          <DumbShareByEmail
+            currentRecipients={recipients}
+            document={document}
+            documentType={documentType}
+            sharing={sharing}
+            pendingRecipients={pendingRecipients}
+            onPendingRecipientsChange={onPendingRecipientsChange}
+            selectedOption={selectedOption}
+            onSelectedOptionChange={onSelectedOptionChange}
+            enableCreateContact
+          />
+        )}
+      </div>
+      {showWhoHasAccess && (
+        <WhoHasAccess
+          document={document}
+          documentType={documentType}
+          isOwner={isOwner}
+          onRevoke={onRevoke}
+          onRevokeSelf={onRevokeSelf}
+          recipients={recipients}
+          recipientsToBeConfirmed={recipientsToBeConfirmed}
+          verifyRecipient={verifyRecipient}
+          link={link}
+          permissions={permissions}
+        />
+      )}
+    </div>
+  )
+}
+
+export default SharingContent