cozy-sharing 28.5.0 → 28.7.0

package/CHANGELOG.md

@@ -3,6 +3,21 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [28.7.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@28.6.0...cozy-sharing@28.7.0) (2026-03-12)
+
+### Features
+
+- Allow to change link editing rights directly from the share modal ([57c3e61](https://github.com/cozy/cozy-libs/commit/57c3e6144649d3102c37961b4953219bdfe553a3))
+- Display the Share restriction modal on click on the sharing link ([a875c36](https://github.com/cozy/cozy-libs/commit/a875c36e7b6ea592b25cbc8386a305b1c1dac516))
+
+# [28.6.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@28.5.0...cozy-sharing@28.6.0) (2026-03-11)
+
+### Features
+
+- Add ability to change member permission type in sharing ([76d0dce](https://github.com/cozy/cozy-libs/commit/76d0dce150d8ea7ac7cb12c6e00b340756b6512a))
+- Hide link text in link recipient ([209857f](https://github.com/cozy/cozy-libs/commit/209857fcf01b71e584e49ef8ab3da4a897d408ed))
+- Remove link prop from LinkRecipient component ([7bbf831](https://github.com/cozy/cozy-libs/commit/7bbf831594d7c33357ade79665f4308fd35e2b8e))
+
 # [28.5.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@28.4.4...cozy-sharing@28.5.0) (2026-03-10)
 
 ### Bug Fixes

package/dist/SharingProvider.js

@@ -463,48 +463,143 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
       };
     }());
 
-    _defineProperty(_this2, "shareByLink", /*#__PURE__*/function () {
-      var _ref12 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10(document, options) {
-        var resp;
+    /**
+     * Update a sharing member's read_only field to change permission type
+     * @param {string} sharingId - The ID of the sharing
+     * @param {number} memberIndex - The index of the member in the sharing members array
+     * @param {string} newType - 'one-way' for read-only, 'two-way' for read-write
+     */
+    _defineProperty(_this2, "updateSharingMemberType", /*#__PURE__*/function () {
+      var _ref12 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10(sharingId, memberIndex, newType) {
+        var sharing, member, matchedContacts, contact, readOnly;
         return _regeneratorRuntime.wrap(function _callee10$(_context10) {
           while (1) switch (_context10.prev = _context10.next) {
             case 0:
-              if (!document.driveId) {
+              sharing = _getSharingById(_this2.state, sharingId);
+
+              if (sharing) {
+                _context10.next = 3;
+                break;
+              }
+
+              throw new Error('Sharing not found');
+
+            case 3:
+              member = sharing.attributes.members[memberIndex];
+
+              if (member) {
                 _context10.next = 6;
                 break;
               }
 
-              _context10.next = 3;
+              throw new Error('Member not found');
+
+            case 6:
+              _context10.next = 8;
+              return _this2.props.client.collection('io.cozy.contacts').find({
+                email: {
+                  $elemMatch: {
+                    address: member.email
+                  }
+                },
+                _id: {
+                  $gt: null
+                }
+              }, {
+                indexedFields: ['_id']
+              });
+
+            case 8:
+              matchedContacts = _context10.sent;
+
+              if (matchedContacts.data.length) {
+                _context10.next = 11;
+                break;
+              }
+
+              throw new Error("Contact not found for member ".concat(member.email));
+
+            case 11:
+              contact = matchedContacts.data[0]; // Revoke (API call only, no state dispatch to avoid UI flickering)
+
+              _context10.next = 14;
+              return _this2.sharingCol.revokeRecipient(sharing, memberIndex);
+
+            case 14:
+              // Re-add with the new permission type
+              readOnly = newType === 'one-way';
+              _context10.prev = 15;
+              _context10.next = 18;
+              return _this2.addRecipients({
+                document: sharing,
+                recipients: readOnly ? [] : [contact],
+                readOnlyRecipients: readOnly ? [contact] : []
+              });
+
+            case 18:
+              _context10.next = 24;
+              break;
+
+            case 20:
+              _context10.prev = 20;
+              _context10.t0 = _context10["catch"](15);
+              log.error("Failed to re-add member ".concat(member.email, " after revocation. The member has been revoked but could not be re-added with the new permission type."), _context10.t0);
+              throw _context10.t0;
+
+            case 24:
+            case "end":
+              return _context10.stop();
+          }
+        }, _callee10, null, [[15, 20]]);
+      }));
+
+      return function (_x16, _x17, _x18) {
+        return _ref12.apply(this, arguments);
+      };
+    }());
+
+    _defineProperty(_this2, "shareByLink", /*#__PURE__*/function () {
+      var _ref13 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11(document, options) {
+        var resp;
+        return _regeneratorRuntime.wrap(function _callee11$(_context11) {
+          while (1) switch (_context11.prev = _context11.next) {
+            case 0:
+              if (!document.driveId) {
+                _context11.next = 6;
+                break;
+              }
+
+              _context11.next = 3;
               return _this2.createSharedDriveSharingLink(document, options);
 
             case 3:
-              _context10.t0 = _context10.sent;
-              _context10.next = 9;
+              _context11.t0 = _context11.sent;
+              _context11.next = 9;
               break;
 
             case 6:
-              _context10.next = 8;
+              _context11.next = 8;
               return _this2.permissionCol.createSharingLink(document, options);
 
             case 8:
-              _context10.t0 = _context10.sent;
+              _context11.t0 = _context11.sent;
 
             case 9:
-              resp = _context10.t0;
+              resp = _context11.t0;
 
               _this2.dispatch(addSharingLink(resp.data));
 
-              return _context10.abrupt("return", resp);
+              return _context11.abrupt("return", resp);
 
             case 12:
             case "end":
-              return _context10.stop();
+              return _context11.stop();
           }
-        }, _callee10);
+        }, _callee11);
       }));
 
-      return function (_x16, _x17) {
-        return _ref12.apply(this, arguments);
+      return function (_x19, _x20) {
+        return _ref13.apply(this, arguments);
       };
     }());
 
@@ -531,67 +626,67 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
     });
 
     _defineProperty(_this2, "createSharedDriveSharingLink", /*#__PURE__*/function () {
-      var _ref13 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11(document, options) {
+      var _ref14 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12(document, options) {
         var client, drivePermissionCollection, resp;
-        return _regeneratorRuntime.wrap(function _callee11$(_context11) {
-          while (1) switch (_context11.prev = _context11.next) {
+        return _regeneratorRuntime.wrap(function _callee12$(_context12) {
+          while (1) switch (_context12.prev = _context12.next) {
             case 0:
               client = _this2.props.client;
               drivePermissionCollection = client.collection('io.cozy.permissions', {
                 driveId: document.driveId
               });
-              _context11.next = 4;
+              _context12.next = 4;
               return drivePermissionCollection.createSharingLink(document, options);
 
             case 4:
-              resp = _context11.sent;
-              return _context11.abrupt("return", resp);
+              resp = _context12.sent;
+              return _context12.abrupt("return", resp);
 
             case 6:
             case "end":
-              return _context11.stop();
+              return _context12.stop();
           }
-        }, _callee11);
+        }, _callee12);
       }));
 
-      return function (_x18, _x19) {
-        return _ref13.apply(this, arguments);
+      return function (_x21, _x22) {
+        return _ref14.apply(this, arguments);
       };
     }());
 
     _defineProperty(_this2, "updateSharedDrivePermissions", /*#__PURE__*/function () {
-      var _ref14 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12(document, permissionDocument, updatedPermissions, options) {
+      var _ref15 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13(document, permissionDocument, updatedPermissions, options) {
         var client, expiresAt, password, drivePermissionCollection, resp;
-        return _regeneratorRuntime.wrap(function _callee12$(_context12) {
-          while (1) switch (_context12.prev = _context12.next) {
+        return _regeneratorRuntime.wrap(function _callee13$(_context13) {
+          while (1) switch (_context13.prev = _context13.next) {
             case 0:
               client = _this2.props.client;
               expiresAt = options.expiresAt, password = options.password;
               drivePermissionCollection = client.collection('io.cozy.permissions', {
                 driveId: document.driveId
               });
-              _context12.next = 5;
+              _context13.next = 5;
               return drivePermissionCollection.add(permissionDocument, updatedPermissions, {
                 expiresAt: expiresAt,
                 password: password
               });
 
             case 5:
-              resp = _context12.sent;
+              resp = _context13.sent;
 
               _this2.dispatch(updateSharingLink(resp.data));
 
-              return _context12.abrupt("return", resp);
+              return _context13.abrupt("return", resp);
 
             case 8:
             case "end":
-              return _context12.stop();
+              return _context13.stop();
           }
-        }, _callee12);
+        }, _callee13);
       }));
 
-      return function (_x20, _x21, _x22, _x23) {
-        return _ref14.apply(this, arguments);
+      return function (_x23, _x24, _x25, _x26) {
+        return _ref15.apply(this, arguments);
       };
     }());
 
@@ -607,28 +702,28 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
      * @return {Array}
      */
     _defineProperty(_this2, "updateDocumentPermissions", /*#__PURE__*/function () {
-      var _ref15 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14(document, options) {
+      var _ref16 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15(document, options) {
         var verbs, expiresAt, password, permissions, responses;
-        return _regeneratorRuntime.wrap(function _callee14$(_context14) {
-          while (1) switch (_context14.prev = _context14.next) {
+        return _regeneratorRuntime.wrap(function _callee15$(_context15) {
+          while (1) switch (_context15.prev = _context15.next) {
             case 0:
               verbs = options.verbs, expiresAt = options.expiresAt, password = options.password;
               permissions = _getDocumentPermissions(_this2.state, document.id);
 
               if (!(!permissions || permissions.length === 0)) {
-                _context14.next = 4;
+                _context15.next = 4;
                 break;
               }
 
-              return _context14.abrupt("return", []);
+              return _context15.abrupt("return", []);
 
             case 4:
-              _context14.next = 6;
+              _context15.next = 6;
               return Promise.all(permissions.map( /*#__PURE__*/function () {
-                var _ref16 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13(permissionDocument) {
+                var _ref17 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14(permissionDocument) {
                   var updatedPermissions, resp;
-                  return _regeneratorRuntime.wrap(function _callee13$(_context13) {
-                    while (1) switch (_context13.prev = _context13.next) {
+                  return _regeneratorRuntime.wrap(function _callee14$(_context14) {
+                    while (1) switch (_context14.prev = _context14.next) {
                       case 0:
                         updatedPermissions = permissionDocument.attributes.permissions;
                         Object.keys(updatedPermissions).forEach(function (permType) {
@@ -636,70 +731,73 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
                         });
 
                         if (!document.driveId) {
-                          _context13.next = 8;
+                          _context14.next = 8;
                           break;
                         }
 
-                        _context13.next = 5;
+                        _context14.next = 5;
                         return _this2.updateSharedDrivePermissions(document, permissionDocument, updatedPermissions, options);
 
                       case 5:
-                        _context13.t0 = _context13.sent;
-                        _context13.next = 11;
+                        _context14.t0 = _context14.sent;
+                        _context14.next = 11;
                         break;
 
                       case 8:
-                        _context13.next = 10;
+                        _context14.next = 10;
                         return _this2.permissionCol.add(permissionDocument, updatedPermissions, {
                           expiresAt: expiresAt,
                           password: password
                         });
 
                       case 10:
-                        _context13.t0 = _context13.sent;
+                        _context14.t0 = _context14.sent;
 
                       case 11:
-                        resp = _context13.t0;
-                        return _context13.abrupt("return", resp);
+                        resp = _context14.t0;
 
-                      case 13:
+                        _this2.dispatch(updateSharingLink(resp.data));
+
+                        return _context14.abrupt("return", resp);
+
+                      case 14:
                       case "end":
-                        return _context13.stop();
+                        return _context14.stop();
                     }
-                  }, _callee13);
+                  }, _callee14);
                 }));
 
-                return function (_x26) {
-                  return _ref16.apply(this, arguments);
+                return function (_x29) {
+                  return _ref17.apply(this, arguments);
                 };
               }()));
 
             case 6:
-              responses = _context14.sent;
-              return _context14.abrupt("return", responses);
+              responses = _context15.sent;
+              return _context15.abrupt("return", responses);
 
             case 8:
             case "end":
-              return _context14.stop();
+              return _context15.stop();
           }
-        }, _callee14);
+        }, _callee15);
       }));
 
-      return function (_x24, _x25) {
-        return _ref15.apply(this, arguments);
+      return function (_x27, _x28) {
+        return _ref16.apply(this, arguments);
       };
     }());
 
     _defineProperty(_this2, "revokeSharingLink", /*#__PURE__*/function () {
-      var _ref17 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15(document) {
+      var _ref18 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16(document) {
         var perms;
-        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:
               // Because some duplicate links have been created in the past, we must ensure
               // we revoke all of them
               perms = _getDocumentPermissions(_this2.state, document.id);
-              _context15.next = 3;
+              _context16.next = 3;
               return Promise.all(perms.map(function (p) {
                 return _this2.permissionCol.destroy(p);
               }));
@@ -709,13 +807,13 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
 
             case 4:
             case "end":
-              return _context15.stop();
+              return _context16.stop();
           }
-        }, _callee15);
+        }, _callee16);
       }));
 
-      return function (_x27) {
-        return _ref17.apply(this, arguments);
+      return function (_x30) {
+        return _ref18.apply(this, arguments);
       };
     }());
 
@@ -802,7 +900,8 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
       revokeAllRecipients: _this2.revokeAllRecipients,
       refresh: _this2.fetchAllSharings,
       hasWriteAccess: _this2.hasWriteAccess,
-      renameSharedDrive: _this2.renameSharedDrive
+      renameSharedDrive: _this2.renameSharedDrive,
+      updateSharingMemberType: _this2.updateSharingMemberType
     };
     _this2.isPublic = (_props$isPublic = props.isPublic) !== null && _props$isPublic !== void 0 ? _props$isPublic : false;
     _this2.realtime = null;
@@ -838,14 +937,14 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
   }, {
     key: "fetchAllSharings",
     value: function () {
-      var _fetchAllSharings = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16() {
+      var _fetchAllSharings = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee17() {
         var _this$props, doctype, client, _yield$Promise$all, _yield$Promise$all2, sharings, permissions, apps, sharedDocIds, resp, folderPaths, filePaths;
 
-        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:
               if (!this.isPublic) {
-                _context16.next = 4;
+                _context17.next = 4;
                 break;
               }
 
@@ -854,17 +953,17 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
               this.setState({
                 allLoaded: true
               });
-              return _context16.abrupt("return");
+              return _context17.abrupt("return");
 
             case 4:
               _this$props = this.props, doctype = _this$props.doctype, client = _this$props.client;
-              _context16.next = 7;
+              _context17.next = 7;
               return Promise.all([this.sharingCol.findByDoctype(doctype, {
                 withSharedDocs: false
               }), this.permissionCol.findLinksByDoctype(doctype), client.fetchQueryAndGetFromState(fetchApps())]);
 
             case 7:
-              _yield$Promise$all = _context16.sent;
+              _yield$Promise$all = _context17.sent;
               _yield$Promise$all2 = _slicedToArray(_yield$Promise$all, 3);
               sharings = _yield$Promise$all2[0];
               permissions = _yield$Promise$all2[1];
@@ -879,35 +978,35 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
                 hasLoadedAtLeastOnePage: true
               }); // eslint-disable-next-line promise/catch-or-return
 
-              _context16.next = 16;
+              _context17.next = 16;
               return fetchNextPermissions(permissions, this.dispatch, this.permissionCol);
 
             case 16:
               if (!(doctype === 'io.cozy.files')) {
-                _context16.next = 26;
+                _context17.next = 26;
                 break;
               }
 
               sharedDocIds = getSharedDocIdsBySharings(sharings);
-              _context16.next = 20;
+              _context17.next = 20;
               return client.collection(doctype).all({
                 keys: sharedDocIds
               });
 
             case 20:
-              resp = _context16.sent;
+              resp = _context17.sent;
               folderPaths = resp.data.filter(function (f) {
                 return f.type === 'directory' && !f.trashed;
               }).map(function (f) {
                 return f.path;
               });
-              _context16.next = 24;
+              _context17.next = 24;
               return fetchFilesPaths(client, doctype, resp.data.filter(function (f) {
                 return f.type !== 'directory' && !f.trashed;
               }));
 
             case 24:
-              filePaths = _context16.sent;
+              filePaths = _context17.sent;
               this.dispatch(receivePaths([].concat(_toConsumableArray(folderPaths), _toConsumableArray(filePaths))));
 
             case 26:
@@ -917,9 +1016,9 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
 
             case 27:
             case "end":
-              return _context16.stop();
+              return _context17.stop();
           }
-        }, _callee16, this);
+        }, _callee17, this);
       }));
 
       function fetchAllSharings() {