cozy-sharing 38.1.0 → 38.1.2

package/CHANGELOG.md

@@ -3,6 +3,19 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [38.1.2](https://github.com/cozy/cozy-libs/compare/cozy-sharing@38.1.1...cozy-sharing@38.1.2) (2026-09-23)
+
+### Bug Fixes
+
+- **sharing:** Reuse the ancestor drive sharing for nested documents ([698048c](https://github.com/cozy/cozy-libs/commit/698048c808fc5f244a57ced6498cb0a9f6562df0))
+
+## [38.1.1](https://github.com/cozy/cozy-libs/compare/cozy-sharing@38.1.0...cozy-sharing@38.1.1) (2026-09-21)
+
+### Bug Fixes
+
+- **sharing:** Keep full share modal for nested-folder editors ([014c848](https://github.com/cozy/cozy-libs/commit/014c848bb46699cee47eb4baf8bb7a365edaf2f1))
+- **sharing:** Show editable share modal for members with write access ([cba3a13](https://github.com/cozy/cozy-libs/commit/cba3a13185d1fe088136c3a758ec94f0d3472b69))
+
 # [38.1.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@38.0.0...cozy-sharing@38.1.0) (2026-09-16)
 
 ### Features

package/dist/SharingProvider.js

@@ -136,8 +136,9 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
           while (1) switch (_context2.prev = _context2.next) {
             case 0:
               document = _ref2.document, recipients = _ref2.recipients, readOnlyRecipients = _ref2.readOnlyRecipients, description = _ref2.description, openSharing = _ref2.openSharing, sharedDrive = _ref2.sharedDrive;
-              _this2$props2 = _this2.props, client = _this2$props2.client, doctype = _this2$props2.doctype;
-              sharing = getDocumentSharing(_this2.state, document.id);
+              _this2$props2 = _this2.props, client = _this2$props2.client, doctype = _this2$props2.doctype; // Documents nested in a shared drive have no sharing of their own
+
+              sharing = getDocumentSharing(_this2.state, document.id) || (document.driveId ? _getSharingById(_this2.state, document.driveId) : null);
 
               if (!sharing) {
                 _context2.next = 12;

package/dist/SharingProvider.spec.js

@@ -793,6 +793,44 @@ describe('shared drive recipient revocation', function () {
       }
     }, _callee14);
   })));
+  it('adds recipients to the existing shared drive when sharing a document nested inside it', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15() {
+    var nestedDocument;
+    return _regeneratorRuntime.wrap(function _callee15$(_context15) {
+      while (1) switch (_context15.prev = _context15.next) {
+        case 0:
+          sharingCol.addRecipients.mockResolvedValue({
+            data: SHARED_DRIVE
+          });
+          nestedDocument = {
+            _id: 'nested_folder_456',
+            id: 'nested_folder_456',
+            driveId: SHARED_DRIVE.id,
+            path: '/Shared folder/Sub folder'
+          };
+          _context15.next = 4;
+          return provider.share({
+            document: nestedDocument,
+            recipients: [recipient],
+            readOnlyRecipients: [],
+            description: 'Sub folder',
+            openSharing: false,
+            sharedDrive: true
+          });
+
+        case 4:
+          expect(sharingCol.create).not.toHaveBeenCalled();
+          expect(sharingCol.addRecipients).toHaveBeenCalledWith({
+            document: SHARED_DRIVE,
+            recipients: [recipient],
+            readOnlyRecipients: []
+          });
+
+        case 6:
+        case "end":
+          return _context15.stop();
+      }
+    }, _callee15);
+  })));
 });
 describe('fetchSharedDriveSharingLinks', function () {
   var PERM_DRIVE_FILE = {
@@ -817,10 +855,10 @@ describe('fetchSharedDriveSharingLinks', function () {
     id: 'file_in_drive',
     driveId: 'drive_123'
   };
-  it('fetches shared drive sharing links by file id', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15() {
+  it('fetches shared drive sharing links by file id', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16() {
     var mockFindLinksByIds, mockClient, provider, result, permissions;
-    return _regeneratorRuntime.wrap(function _callee15$(_context15) {
-      while (1) switch (_context15.prev = _context15.next) {
+    return _regeneratorRuntime.wrap(function _callee16$(_context16) {
+      while (1) switch (_context16.prev = _context16.next) {
         case 0:
           mockFindLinksByIds = jest.fn().mockResolvedValue({
             data: [PERM_DRIVE_FILE]
@@ -830,11 +868,11 @@ describe('fetchSharedDriveSharingLinks', function () {
             findLinksByIds: mockFindLinksByIds
           });
           provider = setupProvider(mockClient);
-          _context15.next = 6;
+          _context16.next = 6;
           return provider.fetchSharedDriveSharingLinks(driveFile);
 
         case 6:
-          result = _context15.sent;
+          result = _context16.sent;
           expect(mockClient.collection).toHaveBeenCalledWith('io.cozy.permissions', {
             driveId: 'drive_123'
           });
@@ -847,14 +885,14 @@ describe('fetchSharedDriveSharingLinks', function () {
 
         case 14:
         case "end":
-          return _context15.stop();
+          return _context16.stop();
       }
-    }, _callee15);
+    }, _callee16);
   })));
-  it('returns an empty array and skips the API call for non-shared-drive documents', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16() {
+  it('returns an empty array and skips the API call for non-shared-drive documents', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee17() {
     var mockClient, findLinksByIds, provider, result;
-    return _regeneratorRuntime.wrap(function _callee16$(_context16) {
-      while (1) switch (_context16.prev = _context16.next) {
+    return _regeneratorRuntime.wrap(function _callee17$(_context17) {
+      while (1) switch (_context17.prev = _context17.next) {
         case 0:
           mockClient = createMockClient({});
           findLinksByIds = jest.fn();
@@ -863,27 +901,27 @@ describe('fetchSharedDriveSharingLinks', function () {
           });
           provider = setupProvider(mockClient);
           provider.dispatch = jest.fn();
-          _context16.next = 7;
+          _context17.next = 7;
           return provider.fetchSharedDriveSharingLinks({
             _id: 'regular_file'
           });
 
         case 7:
-          result = _context16.sent;
+          result = _context17.sent;
           expect(result).toEqual([]);
           expect(findLinksByIds).not.toHaveBeenCalled();
           expect(provider.dispatch).not.toHaveBeenCalled();
 
         case 11:
         case "end":
-          return _context16.stop();
+          return _context17.stop();
       }
-    }, _callee16);
+    }, _callee17);
   })));
-  it('uses document.id as a fallback when _id is missing', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee17() {
+  it('uses document.id as a fallback when _id is missing', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee18() {
     var mockFindLinksByIds, mockClient, provider;
-    return _regeneratorRuntime.wrap(function _callee17$(_context17) {
-      while (1) switch (_context17.prev = _context17.next) {
+    return _regeneratorRuntime.wrap(function _callee18$(_context18) {
+      while (1) switch (_context18.prev = _context18.next) {
         case 0:
           mockFindLinksByIds = jest.fn().mockResolvedValue({
             data: [PERM_DRIVE_FILE]
@@ -893,7 +931,7 @@ describe('fetchSharedDriveSharingLinks', function () {
             findLinksByIds: mockFindLinksByIds
           });
           provider = setupProvider(mockClient);
-          _context17.next = 6;
+          _context18.next = 6;
           return provider.fetchSharedDriveSharingLinks({
             id: 'file_in_drive',
             driveId: 'drive_123'
@@ -904,14 +942,14 @@ describe('fetchSharedDriveSharingLinks', function () {
 
         case 7:
         case "end":
-          return _context17.stop();
+          return _context18.stop();
       }
-    }, _callee17);
+    }, _callee18);
   })));
-  it('does not dispatch when the response has no data', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee18() {
+  it('does not dispatch when the response has no data', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee19() {
     var mockFindLinksByIds, mockClient, provider, result;
-    return _regeneratorRuntime.wrap(function _callee18$(_context18) {
-      while (1) switch (_context18.prev = _context18.next) {
+    return _regeneratorRuntime.wrap(function _callee19$(_context19) {
+      while (1) switch (_context19.prev = _context19.next) {
         case 0:
           mockFindLinksByIds = jest.fn().mockResolvedValue({
             data: []
@@ -922,19 +960,19 @@ describe('fetchSharedDriveSharingLinks', function () {
           });
           provider = setupProvider(mockClient);
           provider.dispatch = jest.fn();
-          _context18.next = 7;
+          _context19.next = 7;
           return provider.fetchSharedDriveSharingLinks(driveFile);
 
         case 7:
-          result = _context18.sent;
+          result = _context19.sent;
           expect(result).toEqual([]);
           expect(provider.dispatch).not.toHaveBeenCalled();
 
         case 10:
         case "end":
-          return _context18.stop();
+          return _context19.stop();
       }
-    }, _callee18);
+    }, _callee19);
   })));
 });
 describe('shareByLink shared drive 409 recovery', function () {
@@ -960,10 +998,10 @@ describe('shareByLink shared drive 409 recovery', function () {
     id: 'file_in_drive',
     driveId: 'drive_123'
   };
-  it('falls back to fetching existing links when create returns 409', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee19() {
+  it('falls back to fetching existing links when create returns 409', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee20() {
     var conflict, mockCreateSharingLink, mockFindLinksByIds, mockClient, provider, resp;
-    return _regeneratorRuntime.wrap(function _callee19$(_context19) {
-      while (1) switch (_context19.prev = _context19.next) {
+    return _regeneratorRuntime.wrap(function _callee20$(_context20) {
+      while (1) switch (_context20.prev = _context20.next) {
         case 0:
           conflict = Object.assign(new Error('Conflict'), {
             status: 409
@@ -978,27 +1016,27 @@ describe('shareByLink shared drive 409 recovery', function () {
             findLinksByIds: mockFindLinksByIds
           });
           provider = setupProvider(mockClient);
-          _context19.next = 8;
+          _context20.next = 8;
           return provider.shareByLink(driveFile, {
             verbs: ['GET']
           });
 
         case 8:
-          resp = _context19.sent;
+          resp = _context20.sent;
           expect(mockCreateSharingLink).toHaveBeenCalled();
           expect(mockFindLinksByIds).toHaveBeenCalledWith(['file_in_drive']);
           expect(resp.data.id).toBe(PERM_DRIVE_FILE.id);
 
         case 12:
         case "end":
-          return _context19.stop();
+          return _context20.stop();
       }
-    }, _callee19);
+    }, _callee20);
   })));
-  it('does not duplicate existing links when 409 recovery refetches them', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee20() {
+  it('does not duplicate existing links when 409 recovery refetches them', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee21() {
     var conflict, mockCreateSharingLink, mockFindLinksByIds, mockClient, provider, resp, permissions;
-    return _regeneratorRuntime.wrap(function _callee20$(_context20) {
-      while (1) switch (_context20.prev = _context20.next) {
+    return _regeneratorRuntime.wrap(function _callee21$(_context21) {
+      while (1) switch (_context21.prev = _context21.next) {
         case 0:
           conflict = Object.assign(new Error('Conflict'), {
             status: 409
@@ -1013,13 +1051,13 @@ describe('shareByLink shared drive 409 recovery', function () {
             findLinksByIds: mockFindLinksByIds
           });
           provider = setupProvider(mockClient, {}, reducer(undefined, addSharingLink(PERM_DRIVE_FILE)));
-          _context20.next = 8;
+          _context21.next = 8;
           return provider.shareByLink(driveFile, {
             verbs: ['GET']
           });
 
         case 8:
-          resp = _context20.sent;
+          resp = _context21.sent;
           permissions = getDocumentPermissions(provider.state, 'file_in_drive');
           expect(resp.data.id).toBe(PERM_DRIVE_FILE.id);
           expect(provider.dispatch).not.toHaveBeenCalled();
@@ -1027,14 +1065,14 @@ describe('shareByLink shared drive 409 recovery', function () {
 
         case 13:
         case "end":
-          return _context20.stop();
+          return _context21.stop();
       }
-    }, _callee20);
+    }, _callee21);
   })));
-  it('rethrows the original error when 409 happens and no link is found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee21() {
+  it('rethrows the original error when 409 happens and no link is found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee22() {
     var conflict, mockCreateSharingLink, mockFindLinksByIds, mockClient, provider;
-    return _regeneratorRuntime.wrap(function _callee21$(_context21) {
-      while (1) switch (_context21.prev = _context21.next) {
+    return _regeneratorRuntime.wrap(function _callee22$(_context22) {
+      while (1) switch (_context22.prev = _context22.next) {
         case 0:
           conflict = Object.assign(new Error('Conflict'), {
             status: 409
@@ -1050,16 +1088,16 @@ describe('shareByLink shared drive 409 recovery', function () {
           });
           provider = setupProvider(mockClient);
           provider.dispatch = jest.fn();
-          _context21.next = 9;
+          _context22.next = 9;
           return expect(provider.shareByLink(driveFile, {
             verbs: ['GET']
           })).rejects.toBe(conflict);
 
         case 9:
         case "end":
-          return _context21.stop();
+          return _context22.stop();
       }
-    }, _callee21);
+    }, _callee22);
   })));
 });
 describe('updateSharingMemberType', function () {
@@ -1102,25 +1140,12 @@ describe('updateSharingMemberType', function () {
       })
     };
   });
-  it('should throw when sharing is not found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee22() {
-    return _regeneratorRuntime.wrap(function _callee22$(_context22) {
-      while (1) switch (_context22.prev = _context22.next) {
-        case 0:
-          _context22.next = 2;
-          return expect(instance.updateSharingMemberType('unknown-id', 1, 'one-way')).rejects.toThrow('Sharing not found');
-
-        case 2:
-        case "end":
-          return _context22.stop();
-      }
-    }, _callee22);
-  })));
-  it('should throw when member is not found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee23() {
+  it('should throw when sharing is not found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee23() {
     return _regeneratorRuntime.wrap(function _callee23$(_context23) {
       while (1) switch (_context23.prev = _context23.next) {
         case 0:
           _context23.next = 2;
-          return expect(instance.updateSharingMemberType('sharing-123', 99, 'one-way')).rejects.toThrow('Member not found');
+          return expect(instance.updateSharingMemberType('unknown-id', 1, 'one-way')).rejects.toThrow('Sharing not found');
 
         case 2:
         case "end":
@@ -1128,33 +1153,29 @@ describe('updateSharingMemberType', function () {
       }
     }, _callee23);
   })));
-  it('should call setReadOnly when switching to one-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee24() {
+  it('should throw when member is not found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee24() {
     return _regeneratorRuntime.wrap(function _callee24$(_context24) {
       while (1) switch (_context24.prev = _context24.next) {
         case 0:
           _context24.next = 2;
-          return instance.updateSharingMemberType('sharing-123', 1, 'one-way');
+          return expect(instance.updateSharingMemberType('sharing-123', 99, 'one-way')).rejects.toThrow('Member not found');
 
         case 2:
-          expect(instance.sharingCol.setReadOnly).toHaveBeenCalledWith(mockSharing, 1);
-          expect(instance.sharingCol.setReadWrite).not.toHaveBeenCalled();
-
-        case 4:
         case "end":
           return _context24.stop();
       }
     }, _callee24);
   })));
-  it('should call setReadWrite when switching to two-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee25() {
+  it('should call setReadOnly when switching to one-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee25() {
     return _regeneratorRuntime.wrap(function _callee25$(_context25) {
       while (1) switch (_context25.prev = _context25.next) {
         case 0:
           _context25.next = 2;
-          return instance.updateSharingMemberType('sharing-123', 1, 'two-way');
+          return instance.updateSharingMemberType('sharing-123', 1, 'one-way');
 
         case 2:
-          expect(instance.sharingCol.setReadWrite).toHaveBeenCalledWith(mockSharing, 1);
-          expect(instance.sharingCol.setReadOnly).not.toHaveBeenCalled();
+          expect(instance.sharingCol.setReadOnly).toHaveBeenCalledWith(mockSharing, 1);
+          expect(instance.sharingCol.setReadWrite).not.toHaveBeenCalled();
 
         case 4:
         case "end":
@@ -1162,37 +1183,29 @@ describe('updateSharingMemberType', function () {
       }
     }, _callee25);
   })));
-  it('should optimistically dispatch a state update when switching to one-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee26() {
+  it('should call setReadWrite when switching to two-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee26() {
     return _regeneratorRuntime.wrap(function _callee26$(_context26) {
       while (1) switch (_context26.prev = _context26.next) {
         case 0:
           _context26.next = 2;
-          return instance.updateSharingMemberType('sharing-123', 1, 'one-way');
+          return instance.updateSharingMemberType('sharing-123', 1, 'two-way');
 
         case 2:
-          expect(instance.dispatch).toHaveBeenCalledWith({
-            type: 'UPDATE_SHARING',
-            sharing: _objectSpread(_objectSpread({}, mockSharing), {}, {
-              attributes: _objectSpread(_objectSpread({}, mockSharing.attributes), {}, {
-                members: [mockSharing.attributes.members[0], _objectSpread(_objectSpread({}, mockSharing.attributes.members[1]), {}, {
-                  read_only: true
-                })]
-              })
-            })
-          });
+          expect(instance.sharingCol.setReadWrite).toHaveBeenCalledWith(mockSharing, 1);
+          expect(instance.sharingCol.setReadOnly).not.toHaveBeenCalled();
 
-        case 3:
+        case 4:
         case "end":
           return _context26.stop();
       }
     }, _callee26);
   })));
-  it('should optimistically dispatch a state update when switching to two-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee27() {
+  it('should optimistically dispatch a state update when switching to one-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee27() {
     return _regeneratorRuntime.wrap(function _callee27$(_context27) {
       while (1) switch (_context27.prev = _context27.next) {
         case 0:
           _context27.next = 2;
-          return instance.updateSharingMemberType('sharing-123', 1, 'two-way');
+          return instance.updateSharingMemberType('sharing-123', 1, 'one-way');
 
         case 2:
           expect(instance.dispatch).toHaveBeenCalledWith({
@@ -1200,7 +1213,7 @@ describe('updateSharingMemberType', function () {
             sharing: _objectSpread(_objectSpread({}, mockSharing), {}, {
               attributes: _objectSpread(_objectSpread({}, mockSharing.attributes), {}, {
                 members: [mockSharing.attributes.members[0], _objectSpread(_objectSpread({}, mockSharing.attributes.members[1]), {}, {
-                  read_only: false
+                  read_only: true
                 })]
               })
             })
@@ -1212,13 +1225,24 @@ describe('updateSharingMemberType', function () {
       }
     }, _callee27);
   })));
-  it('should rethrow error if setReadOnly fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee28() {
+  it('should optimistically dispatch a state update when switching to two-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee28() {
     return _regeneratorRuntime.wrap(function _callee28$(_context28) {
       while (1) switch (_context28.prev = _context28.next) {
         case 0:
-          instance.sharingCol.setReadOnly.mockRejectedValue(new Error('Network error'));
-          _context28.next = 3;
-          return expect(instance.updateSharingMemberType('sharing-123', 1, 'one-way')).rejects.toThrow('Network error');
+          _context28.next = 2;
+          return instance.updateSharingMemberType('sharing-123', 1, 'two-way');
+
+        case 2:
+          expect(instance.dispatch).toHaveBeenCalledWith({
+            type: 'UPDATE_SHARING',
+            sharing: _objectSpread(_objectSpread({}, mockSharing), {}, {
+              attributes: _objectSpread(_objectSpread({}, mockSharing.attributes), {}, {
+                members: [mockSharing.attributes.members[0], _objectSpread(_objectSpread({}, mockSharing.attributes.members[1]), {}, {
+                  read_only: false
+                })]
+              })
+            })
+          });
 
         case 3:
         case "end":
@@ -1226,13 +1250,13 @@ describe('updateSharingMemberType', function () {
       }
     }, _callee28);
   })));
-  it('should rethrow error if setReadWrite fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee29() {
+  it('should rethrow error if setReadOnly fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee29() {
     return _regeneratorRuntime.wrap(function _callee29$(_context29) {
       while (1) switch (_context29.prev = _context29.next) {
         case 0:
-          instance.sharingCol.setReadWrite.mockRejectedValue(new Error('Network error'));
+          instance.sharingCol.setReadOnly.mockRejectedValue(new Error('Network error'));
           _context29.next = 3;
-          return expect(instance.updateSharingMemberType('sharing-123', 1, 'two-way')).rejects.toThrow('Network error');
+          return expect(instance.updateSharingMemberType('sharing-123', 1, 'one-way')).rejects.toThrow('Network error');
 
         case 3:
         case "end":
@@ -1240,12 +1264,26 @@ describe('updateSharingMemberType', function () {
       }
     }, _callee29);
   })));
-  it('should rollback optimistic state update if API call fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee30() {
+  it('should rethrow error if setReadWrite fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee30() {
     return _regeneratorRuntime.wrap(function _callee30$(_context30) {
       while (1) switch (_context30.prev = _context30.next) {
         case 0:
-          instance.sharingCol.setReadOnly.mockRejectedValue(new Error('Network error'));
+          instance.sharingCol.setReadWrite.mockRejectedValue(new Error('Network error'));
           _context30.next = 3;
+          return expect(instance.updateSharingMemberType('sharing-123', 1, 'two-way')).rejects.toThrow('Network error');
+
+        case 3:
+        case "end":
+          return _context30.stop();
+      }
+    }, _callee30);
+  })));
+  it('should rollback optimistic state update if API call fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee31() {
+    return _regeneratorRuntime.wrap(function _callee31$(_context31) {
+      while (1) switch (_context31.prev = _context31.next) {
+        case 0:
+          instance.sharingCol.setReadOnly.mockRejectedValue(new Error('Network error'));
+          _context31.next = 3;
           return expect(instance.updateSharingMemberType('sharing-123', 1, 'one-way')).rejects.toThrow('Network error');
 
         case 3:
@@ -1284,14 +1322,14 @@ describe('updateSharingMemberType', function () {
 
         case 7:
         case "end":
-          return _context30.stop();
+          return _context31.stop();
       }
-    }, _callee30);
+    }, _callee31);
   })));
-  it('should preserve realtime changes when rolling back an API failure', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee32() {
+  it('should preserve realtime changes when rolling back an API failure', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee33() {
     var realtimeSharing;
-    return _regeneratorRuntime.wrap(function _callee32$(_context32) {
-      while (1) switch (_context32.prev = _context32.next) {
+    return _regeneratorRuntime.wrap(function _callee33$(_context33) {
+      while (1) switch (_context33.prev = _context33.next) {
         case 0:
           realtimeSharing = _objectSpread(_objectSpread({}, mockSharing), {}, {
             attributes: _objectSpread(_objectSpread({}, mockSharing.attributes), {}, {
@@ -1303,9 +1341,9 @@ describe('updateSharingMemberType', function () {
               })]
             })
           });
-          instance.sharingCol.setReadOnly.mockImplementation( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee31() {
-            return _regeneratorRuntime.wrap(function _callee31$(_context31) {
-              while (1) switch (_context31.prev = _context31.next) {
+          instance.sharingCol.setReadOnly.mockImplementation( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee32() {
+            return _regeneratorRuntime.wrap(function _callee32$(_context32) {
+              while (1) switch (_context32.prev = _context32.next) {
                 case 0:
                   instance.state = _objectSpread(_objectSpread({}, instance.state), {}, {
                     sharings: instance.state.sharings.map(function (sharing) {
@@ -1316,11 +1354,11 @@ describe('updateSharingMemberType', function () {
 
                 case 2:
                 case "end":
-                  return _context31.stop();
+                  return _context32.stop();
               }
-            }, _callee31);
+            }, _callee32);
           })));
-          _context32.next = 4;
+          _context33.next = 4;
           return expect(instance.updateSharingMemberType('sharing-123', 1, 'one-way')).rejects.toThrow('Network error');
 
         case 4:
@@ -1349,9 +1387,9 @@ describe('updateSharingMemberType', function () {
 
         case 7:
         case "end":
-          return _context32.stop();
+          return _context33.stop();
       }
-    }, _callee32);
+    }, _callee33);
   })));
 }); // TODO Convert with react-testing-library
 // describe('hasWriteAccess', () => {

package/dist/components/FederatedFolder/FederatedFolderModal.js

@@ -64,6 +64,7 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
 
   var _useSharingContext = useSharingContext(),
       canReshare = _useSharingContext.canReshare,
+      hasWriteAccess = _useSharingContext.hasWriteAccess,
       share = _useSharingContext.share,
       getSharingById = _useSharingContext.getSharingById,
       getSharingLink = _useSharingContext.getSharingLink,
@@ -125,7 +126,7 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
   }));
   var isInsideSharedDrive = Boolean((existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument.driveId) && !isSharedDriveRoot);
   var isCurrentUserOwner = existingDocument !== null && existingDocument !== void 0 && existingDocument.driveId ? Boolean(sharedDriveSharing === null || sharedDriveSharing === void 0 || (_sharedDriveSharing$a2 = sharedDriveSharing.attributes) === null || _sharedDriveSharing$a2 === void 0 ? void 0 : _sharedDriveSharing$a2.owner) : documentId ? isOwner(documentId) : false;
-  var isMemberReadOnly = isInsideSharedDrive && !isCurrentUserOwner;
+  var isMemberReadOnly = isInsideSharedDrive && !isCurrentUserOwner && !hasWriteAccess(documentId, existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument.driveId);
   var canManageSharing = isCurrentUserOwner || (documentId ? canReshare(documentId) : false);
 
   var handleRevokeSelf = /*#__PURE__*/function () {
@@ -318,7 +319,7 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
       className: "u-ph-2"
     }, /*#__PURE__*/React.createElement(AntivirusAlert, {
       document: isSending ? frozenDoc : existingDocument
-    }), isInsideSharedDrive ? /*#__PURE__*/React.createElement("div", {
+    }), isMemberReadOnly ? /*#__PURE__*/React.createElement("div", {
       className: styles['share-byemail-onlybylink']
     }, t('Files.share.shareByEmail.onlyByLink', {
       type: t("Files.share.shareByEmail.type.".concat((existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument.type) === 'file' ? 'file' : 'folder'))
@@ -354,7 +355,7 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
       documentType: "Files",
       showGenerateLinkButton: showGenerateLinkButton,
       autoOpenShareRestriction: autoOpenShareRestriction
-    }), !isInsideSharedDrive && /*#__PURE__*/React.createElement(Button, {
+    }), !isMemberReadOnly && /*#__PURE__*/React.createElement(Button, {
       variant: "primary",
       label: t('FederatedFolder.share'),
       busy: isSending,