cozy-sharing 33.1.2 → 33.2.1

package/CHANGELOG.md

@@ -3,6 +3,18 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [33.2.1](https://github.com/cozy/cozy-libs/compare/cozy-sharing@33.2.0...cozy-sharing@33.2.1) (2026-06-09)
+
+### Bug Fixes
+
+- **cozy-sharing:** Refresh link state in share modal ([dfb6dd3](https://github.com/cozy/cozy-libs/commit/dfb6dd39e81b13a7da37ab8f5b4480fb645df01e))
+
+# [33.2.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@33.1.2...cozy-sharing@33.2.0) (2026-06-09)
+
+### Features
+
+- **cozy-sharing:** Show progress icon on done button during share ([885a715](https://github.com/cozy/cozy-libs/commit/885a71568a96354e3d962296a205efb05881a5d1))
+
 ## [33.1.2](https://github.com/cozy/cozy-libs/compare/cozy-sharing@33.1.1...cozy-sharing@33.1.2) (2026-06-08)
 
 ### Bug Fixes

package/dist/components/FederatedFolder/FederatedFolderModal.js

@@ -2,7 +2,7 @@ import _asyncToGenerator from "@babel/runtime/helpers/asyncToGenerator";
 import _slicedToArray from "@babel/runtime/helpers/slicedToArray";
 import _regeneratorRuntime from "@babel/runtime/regenerator";
 import PropTypes from 'prop-types';
-import React, { useCallback, useEffect, useMemo, useState } from 'react';
+import React, { useCallback, useState } from 'react';
 import { useClient } from 'cozy-client';
 import Button from 'cozy-ui/transpiled/react/Buttons';
 import { FixedDialog } from 'cozy-ui/transpiled/react/CozyDialogs';
@@ -51,9 +51,6 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
       share = _useSharingContext.share,
       getSharingLink = _useSharingContext.getSharingLink,
       getFederatedShareLink = _useSharingContext.getFederatedShareLink,
-      getDocumentPermissions = _useSharingContext.getDocumentPermissions,
-      getOwner = _useSharingContext.getOwner,
-      getSharingById = _useSharingContext.getSharingById,
       getRecipients = _useSharingContext.getRecipients,
       hasSharedChild = _useSharingContext.hasSharedChild,
       hasSharedParent = _useSharingContext.hasSharedParent,
@@ -66,11 +63,6 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
       showConfirmDialog = _useConfirmDialog.showConfirmDialog,
       closeConfirmDialog = _useConfirmDialog.closeConfirmDialog;
 
-  var _useState = useState(null),
-      _useState2 = _slicedToArray(_useState, 2),
-      sharingLink = _useState2[0],
-      setSharingLink = _useState2[1];
-
   var _usePendingRecipients = usePendingRecipients(),
       pendingRecipients = _usePendingRecipients.pendingRecipients,
       setPendingRecipients = _usePendingRecipients.setPendingRecipients,
@@ -83,20 +75,20 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
   // we do not use the reactive existingDocument and existingRecipients.
 
 
-  var _useState3 = useState(false),
+  var _useState = useState(false),
+      _useState2 = _slicedToArray(_useState, 2),
+      isSending = _useState2[0],
+      setIsSending = _useState2[1];
+
+  var _useState3 = useState(null),
       _useState4 = _slicedToArray(_useState3, 2),
-      isSending = _useState4[0],
-      setIsSending = _useState4[1];
+      frozenDoc = _useState4[0],
+      setFrozenDoc = _useState4[1];
 
   var _useState5 = useState(null),
       _useState6 = _slicedToArray(_useState5, 2),
-      frozenDoc = _useState6[0],
-      setFrozenDoc = _useState6[1];
-
-  var _useState7 = useState(null),
-      _useState8 = _slicedToArray(_useState7, 2),
-      frozenRecipients = _useState8[0],
-      setFrozenRecipients = _useState8[1];
+      frozenRecipients = _useState6[0],
+      setFrozenRecipients = _useState6[1];
 
   var handleCloseRequest = useCallback(function () {
     if (pendingRecipients.length === 0) {
@@ -122,55 +114,6 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
       }))
     });
   }, [closeConfirmDialog, onClose, pendingRecipients.length, showConfirmDialog, t]);
-  var documentPermissions = useMemo(function () {
-    return existingDocument ? getDocumentPermissions(existingDocument._id) : [];
-  }, [existingDocument, getDocumentPermissions]);
-  useEffect(function () {
-    var fetchSharingLink = /*#__PURE__*/function () {
-      var _ref2 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee() {
-        var link;
-        return _regeneratorRuntime.wrap(function _callee$(_context) {
-          while (1) switch (_context.prev = _context.next) {
-            case 0:
-              if (existingDocument) {
-                _context.next = 2;
-                break;
-              }
-
-              return _context.abrupt("return");
-
-            case 2:
-              if (!existingDocument.driveId) {
-                _context.next = 9;
-                break;
-              }
-
-              _context.next = 5;
-              return getFederatedShareLink(existingDocument);
-
-            case 5:
-              link = _context.sent;
-              setSharingLink(link);
-              _context.next = 10;
-              break;
-
-            case 9:
-              setSharingLink(getSharingLink(existingDocument._id));
-
-            case 10:
-            case "end":
-              return _context.stop();
-          }
-        }, _callee);
-      }));
-
-      return function fetchSharingLink() {
-        return _ref2.apply(this, arguments);
-      };
-    }();
-
-    fetchSharingLink();
-  }, [existingDocument, documentPermissions, getFederatedShareLink, getSharingLink, getOwner, getSharingById]);
   var folderName = (existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument.name) || '';
   var documentPath = existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument.path;
   var hasParentRestriction = Boolean((existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument.driveId) || documentPath && hasSharedParent(documentPath));
@@ -178,34 +121,34 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
   var canShareByEmail = !hasParentRestriction && !hasChildRestriction;
 
   var onSend = /*#__PURE__*/function () {
-    var _ref3 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee2() {
+    var _ref2 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee() {
       var contacts, readWriteRecipients, readOnlyRecipients;
-      return _regeneratorRuntime.wrap(function _callee2$(_context2) {
-        while (1) switch (_context2.prev = _context2.next) {
+      return _regeneratorRuntime.wrap(function _callee$(_context) {
+        while (1) switch (_context.prev = _context.next) {
           case 0:
             if (!(isSending || pendingRecipients.length === 0)) {
-              _context2.next = 3;
+              _context.next = 3;
               break;
             }
 
             onClose();
-            return _context2.abrupt("return");
+            return _context.abrupt("return");
 
           case 3:
             setIsSending(true);
             setFrozenDoc(existingDocument);
             setFrozenRecipients(existingRecipients);
-            _context2.prev = 6;
-            _context2.next = 9;
+            _context.prev = 6;
+            _context.next = 9;
             return getOrCreateFromArray(client, pendingRecipients, function (contact) {
               return client.create('io.cozy.contacts', contact);
             });
 
           case 9:
-            contacts = _context2.sent;
+            contacts = _context.sent;
             readWriteRecipients = selectedOption === 'readOnly' ? [] : contacts;
             readOnlyRecipients = selectedOption === 'readOnly' ? contacts : [];
-            _context2.next = 14;
+            _context.next = 14;
             return share({
               description: folderName,
               document: existingDocument,
@@ -222,12 +165,12 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
               variant: 'filled'
             });
             onClose();
-            _context2.next = 21;
+            _context.next = 21;
             break;
 
           case 18:
-            _context2.prev = 18;
-            _context2.t0 = _context2["catch"](6);
+            _context.prev = 18;
+            _context.t0 = _context["catch"](6);
             showAlert({
               message: t('FederatedFolder.errorNotification'),
               severity: 'error',
@@ -235,23 +178,24 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
             });
 
           case 21:
-            _context2.prev = 21;
+            _context.prev = 21;
             setIsSending(false);
-            return _context2.finish(21);
+            return _context.finish(21);
 
           case 24:
           case "end":
-            return _context2.stop();
+            return _context.stop();
         }
-      }, _callee2, null, [[6, 18, 21, 24]]);
+      }, _callee, null, [[6, 18, 21, 24]]);
     }));
 
     return function onSend() {
-      return _ref3.apply(this, arguments);
+      return _ref2.apply(this, arguments);
     };
   }();
 
   var existingRecipients = existingDocument ? getRecipients(existingDocument._id) : [];
+  var displayedLink = existingDocument !== null && existingDocument !== void 0 && existingDocument.driveId ? getFederatedShareLink(existingDocument) : getSharingLink(existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument._id);
   var modalTitle = t('FederatedFolder.shareTitle', {
     name: folderName
   });
@@ -296,10 +240,10 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
       documentType: "Files",
       className: "u-w-100",
       onRevoke: revoke,
-      link: sharingLink
+      link: displayedLink
     })),
     actions: /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement(ShareByLink, {
-      link: sharingLink,
+      link: displayedLink,
       document: isSending ? frozenDoc : existingDocument,
       documentType: "Files",
       showGenerateLinkButton: showGenerateLinkButton,
@@ -307,7 +251,7 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
     }), canShareByEmail && /*#__PURE__*/React.createElement(Button, {
       variant: "primary",
       label: t('FederatedFolder.share'),
-      disabled: isSending,
+      busy: isSending,
       onClick: onSend
     }))
   });

package/dist/components/FederatedFolder/FederatedFolderModal.spec.js

@@ -7,7 +7,7 @@ function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (O
 function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? ownKeys(Object(source), !0).forEach(function (key) { _defineProperty(target, key, source[key]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)) : ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } return target; }
 
 import _regeneratorRuntime from "@babel/runtime/regenerator";
-import { fireEvent, render, waitFor } from '@testing-library/react';
+import { act, fireEvent, render, waitFor } from '@testing-library/react';
 import React from 'react';
 import { createMockClient } from 'cozy-client';
 import { FederatedFolderModal } from './FederatedFolderModal';
@@ -15,6 +15,7 @@ import { getOrCreateFromArray } from '../../helpers/contacts';
 import { usePendingRecipients } from '../../hooks/usePendingRecipients';
 import AppLike from '../SharingBanner/test/AppLike';
 var mockShare = jest.fn();
+var mockShareByLink = jest.fn();
 var mockRevoke = jest.fn();
 var mockGetSharingLink = jest.fn();
 var mockGetFederatedShareLink = jest.fn();
@@ -28,10 +29,13 @@ jest.mock('../../hooks/useSharingContext', function () {
     useSharingContext: function useSharingContext() {
       return {
         share: mockShare,
+        shareByLink: mockShareByLink,
         revoke: mockRevoke,
         getSharingLink: mockGetSharingLink,
         getFederatedShareLink: mockGetFederatedShareLink,
         getDocumentPermissions: jest.fn().mockReturnValue([]),
+        getOwner: jest.fn(),
+        getSharingById: jest.fn(),
         getRecipients: mockGetRecipients,
         hasSharedChild: mockHasSharedChild,
         hasSharedParent: mockHasSharedParent
@@ -63,6 +67,11 @@ jest.mock('../../hooks/usePendingRecipients', function () {
     usePendingRecipients: jest.fn()
   };
 });
+jest.mock('../ShareByLink', function () {
+  return function () {
+    return /*#__PURE__*/React.createElement("button", null, "Copy link");
+  };
+});
 var mockDocument = {
   _id: 'folder-123',
   name: 'My Test Folder',
@@ -109,10 +118,25 @@ describe('FederatedFolderModal', function () {
   beforeEach(function () {
     jest.clearAllMocks();
     mockGetSharingLink.mockReturnValue('https://example.com/share/abc123');
-    mockGetFederatedShareLink.mockResolvedValue('https://example.com/share/federated-abc123');
+    mockGetFederatedShareLink.mockReturnValue('https://example.com/share/federated-abc123');
     mockGetRecipients.mockReturnValue([]);
     mockHasSharedChild.mockReturnValue(false);
     mockHasSharedParent.mockReturnValue(false);
+    mockShareByLink.mockResolvedValue({
+      data: {
+        _id: 'permission-id',
+        attributes: {
+          shortcodes: {
+            code: 'abc123'
+          }
+        }
+      }
+    });
+    Object.assign(navigator, {
+      clipboard: {
+        writeText: jest.fn()
+      }
+    });
     client = createTestClient();
     sharingContextValue = createSharingContextValue();
     usePendingRecipients.mockReturnValue({
@@ -190,18 +214,44 @@ describe('FederatedFolderModal', function () {
         }
       }, _callee3);
     })));
-    it('should hide share by email when document is in federated shared folder received by current user', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee4() {
-      var _setup4, findByText, queryByText;
+    it('should show link access as soon as a link is generated', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee4() {
+      var _setup4, findByText, queryByText, rerender;
 
       return _regeneratorRuntime.wrap(function _callee4$(_context4) {
         while (1) switch (_context4.prev = _context4.next) {
           case 0:
-            _setup4 = setup({
+            mockGetSharingLink.mockReturnValue(null);
+            _setup4 = setup(), findByText = _setup4.findByText, queryByText = _setup4.queryByText, rerender = _setup4.rerender;
+            expect(queryByText('Anyone with the link')).toBe(null);
+            mockGetSharingLink.mockReturnValue('https://example.com/share/abc123');
+            rerender( /*#__PURE__*/React.createElement(AppLike, {
+              client: client,
+              sharingContextValue: sharingContextValue
+            }, /*#__PURE__*/React.createElement(FederatedFolderModal, {
+              document: mockDocument,
+              onClose: mockOnClose
+            })));
+            _context4.next = 7;
+            return findByText('Anyone with the link');
+
+          case 7:
+          case "end":
+            return _context4.stop();
+        }
+      }, _callee4);
+    })));
+    it('should hide share by email when document is in federated shared folder received by current user', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee5() {
+      var _setup5, findByText, queryByText;
+
+      return _regeneratorRuntime.wrap(function _callee5$(_context5) {
+        while (1) switch (_context5.prev = _context5.next) {
+          case 0:
+            _setup5 = setup({
               document: _objectSpread(_objectSpread({}, mockDocument), {}, {
                 driveId: 'federated-folder-id'
               })
-            }), findByText = _setup4.findByText, queryByText = _setup4.queryByText;
-            _context4.next = 3;
+            }), findByText = _setup5.findByText, queryByText = _setup5.queryByText;
+            _context5.next = 3;
             return findByText('Copy link');
 
           case 3:
@@ -210,34 +260,34 @@ describe('FederatedFolderModal', function () {
 
           case 5:
           case "end":
-            return _context4.stop();
+            return _context5.stop();
         }
-      }, _callee4);
+      }, _callee5);
     })));
-    it('should show only by link message when document is in federated shared folder received by current user', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee5() {
-      var _setup5, findByText;
+    it('should show only by link message when document is in federated shared folder received by current user', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee6() {
+      var _setup6, findByText;
 
-      return _regeneratorRuntime.wrap(function _callee5$(_context5) {
-        while (1) switch (_context5.prev = _context5.next) {
+      return _regeneratorRuntime.wrap(function _callee6$(_context6) {
+        while (1) switch (_context6.prev = _context6.next) {
           case 0:
-            _setup5 = setup({
+            _setup6 = setup({
               document: _objectSpread(_objectSpread({}, mockDocument), {}, {
                 driveId: 'federated-folder-id',
                 type: 'directory'
               })
-            }), findByText = _setup5.findByText;
-            _context5.next = 3;
+            }), findByText = _setup6.findByText;
+            _context6.next = 3;
             return findByText('This folder can only be shared by link, because');
 
           case 3:
-            _context5.next = 5;
+            _context6.next = 5;
             return findByText('it has a shared parent');
 
           case 5:
           case "end":
-            return _context5.stop();
+            return _context6.stop();
         }
-      }, _callee5);
+      }, _callee6);
     })));
   });
   describe('onSend callback', function () {
@@ -245,21 +295,21 @@ describe('FederatedFolderModal', function () {
       name: 'Alice',
       email: 'alice@example.com'
     };
-    it('should call onClose and skip share when there are no pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee6() {
-      var _setup6, findByText, sendButton;
+    it('should call onClose and skip share when there are no pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee7() {
+      var _setup7, findByText, sendButton;
 
-      return _regeneratorRuntime.wrap(function _callee6$(_context6) {
-        while (1) switch (_context6.prev = _context6.next) {
+      return _regeneratorRuntime.wrap(function _callee7$(_context7) {
+        while (1) switch (_context7.prev = _context7.next) {
           case 0:
             // pendingRecipients defaults to [] via beforeEach
-            _setup6 = setup(), findByText = _setup6.findByText;
-            _context6.next = 3;
+            _setup7 = setup(), findByText = _setup7.findByText;
+            _context7.next = 3;
             return findByText('Done');
 
           case 3:
-            sendButton = _context6.sent;
+            sendButton = _context7.sent;
             fireEvent.click(sendButton);
-            _context6.next = 7;
+            _context7.next = 7;
             return waitFor(function () {
               expect(mockOnClose).toHaveBeenCalled();
               expect(mockShare).not.toHaveBeenCalled();
@@ -267,15 +317,15 @@ describe('FederatedFolderModal', function () {
 
           case 7:
           case "end":
-            return _context6.stop();
+            return _context7.stop();
         }
-      }, _callee6);
+      }, _callee7);
     })));
-    it('should call share with correct parameters', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee7() {
-      var _setup7, findByText, sendButton;
+    it('should call share with correct parameters', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee8() {
+      var _setup8, findByText, sendButton;
 
-      return _regeneratorRuntime.wrap(function _callee7$(_context7) {
-        while (1) switch (_context7.prev = _context7.next) {
+      return _regeneratorRuntime.wrap(function _callee8$(_context8) {
+        while (1) switch (_context8.prev = _context8.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [pendingRecipient],
@@ -285,14 +335,14 @@ describe('FederatedFolderModal', function () {
             });
             getOrCreateFromArray.mockResolvedValue([pendingRecipient]);
             mockShare.mockResolvedValueOnce({});
-            _setup7 = setup(), findByText = _setup7.findByText;
-            _context7.next = 6;
+            _setup8 = setup(), findByText = _setup8.findByText;
+            _context8.next = 6;
             return findByText('Done');
 
           case 6:
-            sendButton = _context7.sent;
+            sendButton = _context8.sent;
             fireEvent.click(sendButton);
-            _context7.next = 10;
+            _context8.next = 10;
             return waitFor(function () {
               expect(mockShare).toHaveBeenCalledWith({
                 description: 'My Test Folder',
@@ -306,15 +356,15 @@ describe('FederatedFolderModal', function () {
 
           case 10:
           case "end":
-            return _context7.stop();
+            return _context8.stop();
         }
-      }, _callee7);
+      }, _callee8);
     })));
-    it('should show success alert and close modal on successful share', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee8() {
-      var _setup8, findByText, sendButton;
+    it('should show success alert and close modal on successful share', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee9() {
+      var _setup9, findByText, sendButton;
 
-      return _regeneratorRuntime.wrap(function _callee8$(_context8) {
-        while (1) switch (_context8.prev = _context8.next) {
+      return _regeneratorRuntime.wrap(function _callee9$(_context9) {
+        while (1) switch (_context9.prev = _context9.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [pendingRecipient],
@@ -324,14 +374,14 @@ describe('FederatedFolderModal', function () {
             });
             getOrCreateFromArray.mockResolvedValue([pendingRecipient]);
             mockShare.mockResolvedValueOnce({});
-            _setup8 = setup(), findByText = _setup8.findByText;
-            _context8.next = 6;
+            _setup9 = setup(), findByText = _setup9.findByText;
+            _context9.next = 6;
             return findByText('Done');
 
           case 6:
-            sendButton = _context8.sent;
+            sendButton = _context9.sent;
             fireEvent.click(sendButton);
-            _context8.next = 10;
+            _context9.next = 10;
             return waitFor(function () {
               expect(mockShowAlert).toHaveBeenCalledWith({
                 message: 'Folder has been shared',
@@ -343,15 +393,15 @@ describe('FederatedFolderModal', function () {
 
           case 10:
           case "end":
-            return _context8.stop();
+            return _context9.stop();
         }
-      }, _callee8);
+      }, _callee9);
     })));
-    it('should show error alert when share fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee9() {
-      var _setup9, findByText, sendButton;
+    it('should show error alert when share fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10() {
+      var _setup10, findByText, sendButton;
 
-      return _regeneratorRuntime.wrap(function _callee9$(_context9) {
-        while (1) switch (_context9.prev = _context9.next) {
+      return _regeneratorRuntime.wrap(function _callee10$(_context10) {
+        while (1) switch (_context10.prev = _context10.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [pendingRecipient],
@@ -361,14 +411,14 @@ describe('FederatedFolderModal', function () {
             });
             getOrCreateFromArray.mockResolvedValue([pendingRecipient]);
             mockShare.mockRejectedValueOnce(new Error('Share failed'));
-            _setup9 = setup(), findByText = _setup9.findByText;
-            _context9.next = 6;
+            _setup10 = setup(), findByText = _setup10.findByText;
+            _context10.next = 6;
             return findByText('Done');
 
           case 6:
-            sendButton = _context9.sent;
+            sendButton = _context10.sent;
             fireEvent.click(sendButton);
-            _context9.next = 10;
+            _context10.next = 10;
             return waitFor(function () {
               expect(mockShowAlert).toHaveBeenCalledWith({
                 message: 'Error while sharing folder',
@@ -380,40 +430,92 @@ describe('FederatedFolderModal', function () {
 
           case 10:
           case "end":
-            return _context9.stop();
+            return _context10.stop();
         }
-      }, _callee9);
+      }, _callee10);
+    })));
+    it('should show progress icon on Done button while share is in progress', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12() {
+      var resolveShare, _setup11, findByText, getByRole, queryByRole, sendButton;
+
+      return _regeneratorRuntime.wrap(function _callee12$(_context12) {
+        while (1) switch (_context12.prev = _context12.next) {
+          case 0:
+            usePendingRecipients.mockReturnValue({
+              pendingRecipients: [pendingRecipient],
+              setPendingRecipients: jest.fn(),
+              selectedOption: 'readWrite',
+              setSelectedOption: jest.fn()
+            });
+            getOrCreateFromArray.mockResolvedValue([pendingRecipient]);
+            mockShare.mockReturnValueOnce(new Promise(function (resolve) {
+              resolveShare = resolve;
+            }));
+            _setup11 = setup(), findByText = _setup11.findByText, getByRole = _setup11.getByRole, queryByRole = _setup11.queryByRole;
+            _context12.next = 6;
+            return findByText('Done');
+
+          case 6:
+            sendButton = _context12.sent;
+            fireEvent.click(sendButton);
+            _context12.next = 10;
+            return waitFor(function () {
+              expect(getByRole('button', {
+                name: 'Done'
+              })).toBeDisabled();
+              expect(queryByRole('progressbar')).not.toBeNull();
+            });
+
+          case 10:
+            _context12.next = 12;
+            return act( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11() {
+              return _regeneratorRuntime.wrap(function _callee11$(_context11) {
+                while (1) switch (_context11.prev = _context11.next) {
+                  case 0:
+                    resolveShare({});
+
+                  case 1:
+                  case "end":
+                    return _context11.stop();
+                }
+              }, _callee11);
+            })));
+
+          case 12:
+          case "end":
+            return _context12.stop();
+        }
+      }, _callee12);
     })));
   });
   describe('onClose callback', function () {
-    it('should call onClose when close button is clicked', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10() {
-      var _setup10, findByRole, closeButton;
+    it('should call onClose when close button is clicked', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13() {
+      var _setup12, findByRole, closeButton;
 
-      return _regeneratorRuntime.wrap(function _callee10$(_context10) {
-        while (1) switch (_context10.prev = _context10.next) {
+      return _regeneratorRuntime.wrap(function _callee13$(_context13) {
+        while (1) switch (_context13.prev = _context13.next) {
           case 0:
-            _setup10 = setup(), findByRole = _setup10.findByRole;
-            _context10.next = 3;
+            _setup12 = setup(), findByRole = _setup12.findByRole;
+            _context13.next = 3;
             return findByRole('button', {
               name: /close/i
             });
 
           case 3:
-            closeButton = _context10.sent;
+            closeButton = _context13.sent;
             fireEvent.click(closeButton);
             expect(mockOnClose).toHaveBeenCalled();
 
           case 6:
           case "end":
-            return _context10.stop();
+            return _context13.stop();
         }
-      }, _callee10);
+      }, _callee13);
     })));
-    it('should ask confirmation when close button is clicked with pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11() {
-      var _setup11, getByRole, findByRole, getByText, closeButton;
+    it('should ask confirmation when close button is clicked with pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14() {
+      var _setup13, getByRole, findByRole, getByText, closeButton;
 
-      return _regeneratorRuntime.wrap(function _callee11$(_context11) {
-        while (1) switch (_context11.prev = _context11.next) {
+      return _regeneratorRuntime.wrap(function _callee14$(_context14) {
+        while (1) switch (_context14.prev = _context14.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [{
@@ -424,14 +526,14 @@ describe('FederatedFolderModal', function () {
               selectedOption: 'readWrite',
               setSelectedOption: jest.fn()
             });
-            _setup11 = setup(), getByRole = _setup11.getByRole, findByRole = _setup11.findByRole, getByText = _setup11.getByText;
-            _context11.next = 4;
+            _setup13 = setup(), getByRole = _setup13.getByRole, findByRole = _setup13.findByRole, getByText = _setup13.getByText;
+            _context14.next = 4;
             return findByRole('button', {
               name: /close/i
             });
 
           case 4:
-            closeButton = _context11.sent;
+            closeButton = _context14.sent;
             fireEvent.click(closeButton);
             expect(getByText("Discard the changes you haven't saved?")).toBeTruthy();
             expect(mockOnClose).not.toHaveBeenCalled();
@@ -442,15 +544,15 @@ describe('FederatedFolderModal', function () {
 
           case 10:
           case "end":
-            return _context11.stop();
+            return _context14.stop();
         }
-      }, _callee11);
+      }, _callee14);
     })));
-    it('should close after discard confirmation', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12() {
-      var _setup12, getByRole, findByRole, getByText, closeButton;
+    it('should close after discard confirmation', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15() {
+      var _setup14, getByRole, findByRole, getByText, closeButton;
 
-      return _regeneratorRuntime.wrap(function _callee12$(_context12) {
-        while (1) switch (_context12.prev = _context12.next) {
+      return _regeneratorRuntime.wrap(function _callee15$(_context15) {
+        while (1) switch (_context15.prev = _context15.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [{
@@ -461,14 +563,14 @@ describe('FederatedFolderModal', function () {
               selectedOption: 'readWrite',
               setSelectedOption: jest.fn()
             });
-            _setup12 = setup(), getByRole = _setup12.getByRole, findByRole = _setup12.findByRole, getByText = _setup12.getByText;
-            _context12.next = 4;
+            _setup14 = setup(), getByRole = _setup14.getByRole, findByRole = _setup14.findByRole, getByText = _setup14.getByText;
+            _context15.next = 4;
             return findByRole('button', {
               name: /close/i
             });
 
           case 4:
-            closeButton = _context12.sent;
+            closeButton = _context15.sent;
             fireEvent.click(closeButton);
             expect(getByText("Discard the changes you haven't saved?")).toBeTruthy();
             fireEvent.click(getByRole('button', {
@@ -478,9 +580,9 @@ describe('FederatedFolderModal', function () {
 
           case 9:
           case "end":
-            return _context12.stop();
+            return _context15.stop();
         }
-      }, _callee12);
+      }, _callee15);
     })));
   });
 });

package/dist/components/ShareDialogCozyToCozy.js

@@ -24,6 +24,7 @@ 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';
 var styles = {
   "share-modal-content": "share__share-modal-content___1iqEq",
   "coz-form": "share__coz-form___1ICST",
@@ -144,6 +145,9 @@ var ShareDialogCozyToCozy = function ShareDialogCozyToCozy(_ref3) {
   var _useAlert = useAlert(),
       showAlert = _useAlert.showAlert;
 
+  var _useSharingContext = useSharingContext(),
+      getSharingLink = _useSharingContext.getSharingLink;
+
   var _usePendingRecipients = usePendingRecipients(),
       pendingRecipients = _usePendingRecipients.pendingRecipients,
       setPendingRecipients = _usePendingRecipients.setPendingRecipients,
@@ -160,6 +164,7 @@ var ShareDialogCozyToCozy = function ShareDialogCozyToCozy(_ref3) {
       showRecipientsLimit = _useState4[0],
       setShowRecipientsLimit = _useState4[1];
 
+  var displayedLink = getSharingLink(document._id || document.id);
   var handleShare = useCallback( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee() {
     var contacts, readWriteRecipients, readOnlyRecipients;
     return _regeneratorRuntime.wrap(function _callee$(_context) {
@@ -254,6 +259,7 @@ var ShareDialogCozyToCozy = function ShareDialogCozyToCozy(_ref3) {
     return Actions;
   }, [handleShare, submitting, t, documentType, showShareByLink, showGenerateLinkButton, autoOpenShareRestriction]);
   return /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement(ShareDialogTwoStepsConfirmationContainer, _extends({}, props, {
+    link: displayedLink,
     documentType: documentType,
     document: document,
     recipients: recipients,

package/dist/components/ShareDialogCozyToCozy.spec.js

@@ -33,6 +33,9 @@ jest.mock('../hooks/useSharingContext', function () {
   return {
     useSharingContext: function useSharingContext() {
       return {
+        getSharingLink: jest.fn(function () {
+          return null;
+        }),
         updateSharingMemberType: jest.fn()
       };
     }

package/dist/components/ShareDialogOnlyByLink.js

@@ -7,6 +7,7 @@ import { useI18n } from 'twake-i18n';
 import AntivirusAlert from './AntivirusAlert';
 import { default as DumbShareByLink } from './ShareByLink';
 import WhoHasAccess from './WhoHasAccess';
+import { useSharingContext } from '../hooks/useSharingContext';
 
 var ShareDialogOnlyByLink = function ShareDialogOnlyByLink(_ref) {
   var isOwner = _ref.isOwner,
@@ -15,7 +16,6 @@ var ShareDialogOnlyByLink = function ShareDialogOnlyByLink(_ref) {
       recipients = _ref.recipients,
       document = _ref.document,
       documentType = _ref.documentType,
-      link = _ref.link,
       onClose = _ref.onClose,
       permissions = _ref.permissions,
       showGenerateLinkButton = _ref.showGenerateLinkButton,
@@ -27,6 +27,10 @@ var ShareDialogOnlyByLink = function ShareDialogOnlyByLink(_ref) {
   var _useBreakpoints = useBreakpoints(),
       isMobile = _useBreakpoints.isMobile;
 
+  var _useSharingContext = useSharingContext(),
+      getSharingLink = _useSharingContext.getSharingLink;
+
+  var displayedLink = getSharingLink(document._id || document.id);
   return /*#__PURE__*/React.createElement(Dialog, {
     disableGutters: true,
     disableEnforceFocus: true,
@@ -47,12 +51,12 @@ var ShareDialogOnlyByLink = function ShareDialogOnlyByLink(_ref) {
       onRevoke: onRevoke,
       onRevokeSelf: onRevokeSelf,
       recipients: recipients,
-      link: link,
+      link: displayedLink,
       permissions: permissions
     }), /*#__PURE__*/React.createElement("div", {
       className: cx('u-mt-half', isMobile ? 'u-ph-1 u-mb-1' : 'u-ph-2 u-mb-1-half')
     }, /*#__PURE__*/React.createElement(DumbShareByLink, {
-      link: link,
+      link: displayedLink,
       document: document,
       documentType: documentType,
       showGenerateLinkButton: showGenerateLinkButton,

package/dist/components/ShareDialogOnlyByLink.spec.js

@@ -0,0 +1,75 @@
+import { render, screen } from '@testing-library/react';
+import React from 'react';
+import ShareDialogOnlyByLink from './ShareDialogOnlyByLink';
+import AppLike from '../../test/AppLike';
+var mockSharingLink = null;
+jest.mock('./ShareByLink', function () {
+  return function () {
+    return /*#__PURE__*/React.createElement("button", null, "Copy link");
+  };
+});
+jest.mock('../hooks/useSharingContext', function () {
+  return {
+    useSharingContext: function useSharingContext() {
+      return {
+        documentType: 'Files',
+        getDocumentPermissions: function getDocumentPermissions() {
+          return [];
+        },
+        getSharingLink: function getSharingLink() {
+          return mockSharingLink;
+        },
+        revokeSharingLink: jest.fn(),
+        updateDocumentPermissions: jest.fn()
+      };
+    }
+  };
+});
+describe('ShareDialogOnlyByLink', function () {
+  beforeEach(function () {
+    mockSharingLink = null;
+  });
+
+  var renderModal = function renderModal() {
+    return /*#__PURE__*/React.createElement(AppLike, null, /*#__PURE__*/React.createElement(ShareDialogOnlyByLink, {
+      document: {
+        _id: 'file-id',
+        attributes: {
+          name: 'file.txt'
+        }
+      },
+      documentType: "Files",
+      isOwner: true,
+      onClose: jest.fn(),
+      onRevoke: jest.fn(),
+      onRevokeSelf: jest.fn(),
+      permissions: [],
+      recipients: []
+    }));
+  };
+
+  var setup = function setup() {
+    return render(renderModal());
+  };
+
+  it('shows link access as soon as a link is generated', function () {
+    var _setup = setup(),
+        rerender = _setup.rerender;
+
+    expect(screen.queryByText('Anyone with the link')).toBe(null);
+    mockSharingLink = 'https://example.com/public';
+    rerender(renderModal());
+    expect(screen.queryByText('Anyone with the link')).toBeInTheDocument();
+  });
+  it('hides link access as soon as a link is revoked', function () {
+    mockSharingLink = 'https://example.com/public';
+
+    var _setup2 = setup(),
+        rerender = _setup2.rerender;
+
+    expect(screen.queryByText('Anyone with the link')).toBeInTheDocument();
+    mockSharingLink = null;
+    rerender(renderModal());
+    expect(screen.queryByText('Anyone with the link')).toBe(null);
+  });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-sharing",
-  "version": "33.1.2",
+  "version": "33.2.1",
   "description": "Provides sharing login for React applications.",
   "main": "dist/index.js",
   "author": "Cozy",
@@ -83,5 +83,5 @@
   "sideEffects": [
     "*.css"
   ],
-  "gitHead": "4f63c28ae3cd8cac22487e6640006de4f5c31cfc"
+  "gitHead": "107c14be0fb0677515062187e3fdadeab4e00350"
 }

package/src/components/FederatedFolder/FederatedFolderModal.jsx

@@ -1,5 +1,5 @@
 import PropTypes from 'prop-types'
-import React, { useCallback, useEffect, useMemo, useState } from 'react'
+import React, { useCallback, useState } from 'react'
 
 import { useClient } from 'cozy-client'
 import Button from 'cozy-ui/transpiled/react/Buttons'
@@ -33,9 +33,6 @@ const FederatedFolderModalContent = ({
     share,
     getSharingLink,
     getFederatedShareLink,
-    getDocumentPermissions,
-    getOwner,
-    getSharingById,
     getRecipients,
     hasSharedChild,
     hasSharedParent,
@@ -44,7 +41,6 @@ const FederatedFolderModalContent = ({
   const { showAlert } = useAlert()
   const { showConfirmDialog, closeConfirmDialog } = useConfirmDialog()
 
-  const [sharingLink, setSharingLink] = useState(null)
   const {
     pendingRecipients,
     setPendingRecipients,
@@ -98,34 +94,6 @@ const FederatedFolderModalContent = ({
     t
   ])
 
-  const documentPermissions = useMemo(
-    () =>
-      existingDocument ? getDocumentPermissions(existingDocument._id) : [],
-    [existingDocument, getDocumentPermissions]
-  )
-
-  useEffect(() => {
-    const fetchSharingLink = async () => {
-      if (!existingDocument) return
-
-      if (existingDocument.driveId) {
-        const link = await getFederatedShareLink(existingDocument)
-        setSharingLink(link)
-      } else {
-        setSharingLink(getSharingLink(existingDocument._id))
-      }
-    }
-
-    fetchSharingLink()
-  }, [
-    existingDocument,
-    documentPermissions,
-    getFederatedShareLink,
-    getSharingLink,
-    getOwner,
-    getSharingById
-  ])
-
   const folderName = existingDocument?.name || ''
   const documentPath = existingDocument?.path
   const hasParentRestriction = Boolean(
@@ -185,6 +153,9 @@ const FederatedFolderModalContent = ({
   const existingRecipients = existingDocument
     ? getRecipients(existingDocument._id)
     : []
+  const displayedLink = existingDocument?.driveId
+    ? getFederatedShareLink(existingDocument)
+    : getSharingLink(existingDocument?._id)
 
   const modalTitle = t('FederatedFolder.shareTitle', { name: folderName })
 
@@ -252,14 +223,14 @@ const FederatedFolderModalContent = ({
             documentType="Files"
             className="u-w-100"
             onRevoke={revoke}
-            link={sharingLink}
+            link={displayedLink}
           />
         </div>
       }
       actions={
         <>
           <ShareByLink
-            link={sharingLink}
+            link={displayedLink}
             document={isSending ? frozenDoc : existingDocument}
             documentType="Files"
             showGenerateLinkButton={showGenerateLinkButton}
@@ -269,7 +240,7 @@ const FederatedFolderModalContent = ({
             <Button
               variant="primary"
               label={t('FederatedFolder.share')}
-              disabled={isSending}
+              busy={isSending}
               onClick={onSend}
             />
           )}

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

@@ -1,4 +1,4 @@
-import { fireEvent, render, waitFor } from '@testing-library/react'
+import { act, fireEvent, render, waitFor } from '@testing-library/react'
 import React from 'react'
 
 import { createMockClient } from 'cozy-client'
@@ -9,6 +9,7 @@ import { usePendingRecipients } from '../../hooks/usePendingRecipients'
 import AppLike from '../SharingBanner/test/AppLike'
 
 const mockShare = jest.fn()
+const mockShareByLink = jest.fn()
 const mockRevoke = jest.fn()
 const mockGetSharingLink = jest.fn()
 const mockGetFederatedShareLink = jest.fn()
@@ -21,10 +22,13 @@ const mockOnClose = jest.fn()
 jest.mock('../../hooks/useSharingContext', () => ({
   useSharingContext: () => ({
     share: mockShare,
+    shareByLink: mockShareByLink,
     revoke: mockRevoke,
     getSharingLink: mockGetSharingLink,
     getFederatedShareLink: mockGetFederatedShareLink,
     getDocumentPermissions: jest.fn().mockReturnValue([]),
+    getOwner: jest.fn(),
+    getSharingById: jest.fn(),
     getRecipients: mockGetRecipients,
     hasSharedChild: mockHasSharedChild,
     hasSharedParent: mockHasSharedParent
@@ -47,6 +51,8 @@ jest.mock('../../hooks/usePendingRecipients', () => ({
   usePendingRecipients: jest.fn()
 }))
 
+jest.mock('../ShareByLink', () => () => <button>Copy link</button>)
+
 const mockDocument = {
   _id: 'folder-123',
   name: 'My Test Folder',
@@ -90,12 +96,25 @@ describe('FederatedFolderModal', () => {
   beforeEach(() => {
     jest.clearAllMocks()
     mockGetSharingLink.mockReturnValue('https://example.com/share/abc123')
-    mockGetFederatedShareLink.mockResolvedValue(
+    mockGetFederatedShareLink.mockReturnValue(
       'https://example.com/share/federated-abc123'
     )
     mockGetRecipients.mockReturnValue([])
     mockHasSharedChild.mockReturnValue(false)
     mockHasSharedParent.mockReturnValue(false)
+    mockShareByLink.mockResolvedValue({
+      data: {
+        _id: 'permission-id',
+        attributes: {
+          shortcodes: { code: 'abc123' }
+        }
+      }
+    })
+    Object.assign(navigator, {
+      clipboard: {
+        writeText: jest.fn()
+      }
+    })
     client = createTestClient()
     sharingContextValue = createSharingContextValue()
     usePendingRecipients.mockReturnValue({
@@ -139,6 +158,22 @@ describe('FederatedFolderModal', () => {
       await findByText('Copy link')
     })
 
+    it('should show link access as soon as a link is generated', async () => {
+      mockGetSharingLink.mockReturnValue(null)
+      const { findByText, queryByText, rerender } = setup()
+
+      expect(queryByText('Anyone with the link')).toBe(null)
+
+      mockGetSharingLink.mockReturnValue('https://example.com/share/abc123')
+      rerender(
+        <AppLike client={client} sharingContextValue={sharingContextValue}>
+          <FederatedFolderModal document={mockDocument} onClose={mockOnClose} />
+        </AppLike>
+      )
+
+      await findByText('Anyone with the link')
+    })
+
     it('should hide share by email when document is in federated shared folder received by current user', async () => {
       const { findByText, queryByText } = setup({
         document: { ...mockDocument, driveId: 'federated-folder-id' }
@@ -257,6 +292,36 @@ describe('FederatedFolderModal', () => {
         expect(mockOnClose).not.toHaveBeenCalled()
       })
     })
+
+    it('should show progress icon on Done button while share is in progress', async () => {
+      usePendingRecipients.mockReturnValue({
+        pendingRecipients: [pendingRecipient],
+        setPendingRecipients: jest.fn(),
+        selectedOption: 'readWrite',
+        setSelectedOption: jest.fn()
+      })
+      getOrCreateFromArray.mockResolvedValue([pendingRecipient])
+      let resolveShare
+      mockShare.mockReturnValueOnce(
+        new Promise(resolve => {
+          resolveShare = resolve
+        })
+      )
+      const { findByText, getByRole, queryByRole } = setup()
+
+      const sendButton = await findByText('Done')
+
+      fireEvent.click(sendButton)
+
+      await waitFor(() => {
+        expect(getByRole('button', { name: 'Done' })).toBeDisabled()
+        expect(queryByRole('progressbar')).not.toBeNull()
+      })
+
+      await act(async () => {
+        resolveShare({})
+      })
+    })
   })
 
   describe('onClose callback', () => {

package/src/components/ShareDialogCozyToCozy.jsx

@@ -19,6 +19,7 @@ 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 = ({
@@ -126,6 +127,7 @@ const ShareDialogCozyToCozy = ({
   const { t } = useI18n()
   const client = useClient()
   const { showAlert } = useAlert()
+  const { getSharingLink } = useSharingContext()
 
   const {
     pendingRecipients,
@@ -136,6 +138,7 @@ const ShareDialogCozyToCozy = ({
 
   const [submitting, setSubmitting] = useState(false)
   const [showRecipientsLimit, setShowRecipientsLimit] = useState(false)
+  const displayedLink = getSharingLink(document._id || document.id)
 
   const handleShare = useCallback(async () => {
     if (pendingRecipients.length === 0) {
@@ -238,6 +241,7 @@ const ShareDialogCozyToCozy = ({
     <>
       <ShareDialogTwoStepsConfirmationContainer
         {...props}
+        link={displayedLink}
         documentType={documentType}
         document={document}
         recipients={recipients}

package/src/components/ShareDialogCozyToCozy.spec.jsx

@@ -16,6 +16,7 @@ jest.mock('cozy-client', () => ({
 
 jest.mock('../hooks/useSharingContext', () => ({
   useSharingContext: () => ({
+    getSharingLink: jest.fn(() => null),
     updateSharingMemberType: jest.fn()
   })
 }))

package/src/components/ShareDialogOnlyByLink.jsx

@@ -9,6 +9,7 @@ import { useI18n } from 'twake-i18n'
 import AntivirusAlert from './AntivirusAlert'
 import { default as DumbShareByLink } from './ShareByLink'
 import WhoHasAccess from './WhoHasAccess'
+import { useSharingContext } from '../hooks/useSharingContext'
 
 const ShareDialogOnlyByLink = ({
   isOwner,
@@ -17,7 +18,6 @@ const ShareDialogOnlyByLink = ({
   recipients,
   document,
   documentType,
-  link,
   onClose,
   permissions,
   showGenerateLinkButton,
@@ -25,6 +25,8 @@ const ShareDialogOnlyByLink = ({
 }) => {
   const { t } = useI18n()
   const { isMobile } = useBreakpoints()
+  const { getSharingLink } = useSharingContext()
+  const displayedLink = getSharingLink(document._id || document.id)
 
   return (
     <Dialog
@@ -51,7 +53,7 @@ const ShareDialogOnlyByLink = ({
             onRevoke={onRevoke}
             onRevokeSelf={onRevokeSelf}
             recipients={recipients}
-            link={link}
+            link={displayedLink}
             permissions={permissions}
           />
           <div
@@ -61,7 +63,7 @@ const ShareDialogOnlyByLink = ({
             )}
           >
             <DumbShareByLink
-              link={link}
+              link={displayedLink}
               document={document}
               documentType={documentType}
               showGenerateLinkButton={showGenerateLinkButton}

package/src/components/ShareDialogOnlyByLink.spec.jsx

@@ -0,0 +1,65 @@
+import { render, screen } from '@testing-library/react'
+import React from 'react'
+
+import ShareDialogOnlyByLink from './ShareDialogOnlyByLink'
+import AppLike from '../../test/AppLike'
+
+let mockSharingLink = null
+
+jest.mock('./ShareByLink', () => () => <button>Copy link</button>)
+
+jest.mock('../hooks/useSharingContext', () => ({
+  useSharingContext: () => ({
+    documentType: 'Files',
+    getDocumentPermissions: () => [],
+    getSharingLink: () => mockSharingLink,
+    revokeSharingLink: jest.fn(),
+    updateDocumentPermissions: jest.fn()
+  })
+}))
+
+describe('ShareDialogOnlyByLink', () => {
+  beforeEach(() => {
+    mockSharingLink = null
+  })
+
+  const renderModal = () => (
+    <AppLike>
+      <ShareDialogOnlyByLink
+        document={{ _id: 'file-id', attributes: { name: 'file.txt' } }}
+        documentType="Files"
+        isOwner
+        onClose={jest.fn()}
+        onRevoke={jest.fn()}
+        onRevokeSelf={jest.fn()}
+        permissions={[]}
+        recipients={[]}
+      />
+    </AppLike>
+  )
+
+  const setup = () => render(renderModal())
+
+  it('shows link access as soon as a link is generated', () => {
+    const { rerender } = setup()
+
+    expect(screen.queryByText('Anyone with the link')).toBe(null)
+
+    mockSharingLink = 'https://example.com/public'
+    rerender(renderModal())
+
+    expect(screen.queryByText('Anyone with the link')).toBeInTheDocument()
+  })
+
+  it('hides link access as soon as a link is revoked', () => {
+    mockSharingLink = 'https://example.com/public'
+    const { rerender } = setup()
+
+    expect(screen.queryByText('Anyone with the link')).toBeInTheDocument()
+
+    mockSharingLink = null
+    rerender(renderModal())
+
+    expect(screen.queryByText('Anyone with the link')).toBe(null)
+  })
+})