cozy-sharing 28.4.4 → 28.6.0

package/dist/components/ShareRestrictionModal/helpers.js

@@ -3,6 +3,7 @@ import _regeneratorRuntime from "@babel/runtime/regenerator";
 import { generateWebLink } from 'cozy-client';
 import minilog from 'cozy-minilog';
 import { getAppSlugFromDocumentType } from '../../helpers/link';
+import { getShortcode } from '../../helpers/shortcodes';
 var log = minilog('ShareRestrictionModal/helpers');
 export var WRITE_PERMS = ['GET', 'POST', 'PUT', 'PATCH'];
 export var READ_ONLY_PERMS = ['GET'];
@@ -24,14 +25,12 @@ export var READ_ONLY_PERMS = ['GET'];
 
 export var createSharingLink = /*#__PURE__*/function () {
   var _ref2 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee(_ref) {
-    var _permsResult$data;
-
-    var client, file, documentType, shareByLink, t, ttl, _ref$editingRights, editingRights, password, showAlert, permsResult, perms;
+    var client, file, documentType, shareByLink, getOwner, getSharingById, t, ttl, _ref$editingRights, editingRights, password, showAlert, permsResult, code;
 
     return _regeneratorRuntime.wrap(function _callee$(_context) {
       while (1) switch (_context.prev = _context.next) {
         case 0:
-          client = _ref.client, file = _ref.file, documentType = _ref.documentType, shareByLink = _ref.shareByLink, t = _ref.t, ttl = _ref.ttl, _ref$editingRights = _ref.editingRights, editingRights = _ref$editingRights === void 0 ? 'readOnly' : _ref$editingRights, password = _ref.password, showAlert = _ref.showAlert;
+          client = _ref.client, file = _ref.file, documentType = _ref.documentType, shareByLink = _ref.shareByLink, getOwner = _ref.getOwner, getSharingById = _ref.getSharingById, t = _ref.t, ttl = _ref.ttl, _ref$editingRights = _ref.editingRights, editingRights = _ref$editingRights === void 0 ? 'readOnly' : _ref$editingRights, password = _ref.password, showAlert = _ref.showAlert;
           _context.next = 3;
           return createPermissions({
             file: file,
@@ -46,24 +45,23 @@ export var createSharingLink = /*#__PURE__*/function () {
 
         case 3:
           permsResult = _context.sent;
+          code = getShortcode(permsResult === null || permsResult === void 0 ? void 0 : permsResult.data);
 
-          if (permsResult !== null && permsResult !== void 0 && (_permsResult$data = permsResult.data) !== null && _permsResult$data !== void 0 && (_permsResult$data = _permsResult$data.attributes) !== null && _permsResult$data !== void 0 && (_permsResult$data = _permsResult$data.shortcodes) !== null && _permsResult$data !== void 0 && _permsResult$data.code) {
-            _context.next = 6;
+          if (code) {
+            _context.next = 7;
             break;
           }
 
           return _context.abrupt("return", null);
 
-        case 6:
-          perms = permsResult.data;
-          return _context.abrupt("return", generateWebLink({
-            cozyUrl: client.getStackClient().uri,
-            searchParams: [['sharecode', perms.attributes.shortcodes.code]],
-            pathname: '/public',
-            slug: getAppSlugFromDocumentType({
-              documentType: documentType
-            }),
-            subDomainType: client.capabilities.flat_subdomains ? 'flat' : 'nested'
+        case 7:
+          return _context.abrupt("return", generateShareLinkFromFile({
+            client: client,
+            file: file,
+            sharecode: code,
+            getOwner: getOwner,
+            getSharingById: getSharingById,
+            documentType: documentType
           }));
 
         case 8:
@@ -77,6 +75,45 @@ export var createSharingLink = /*#__PURE__*/function () {
     return _ref2.apply(this, arguments);
   };
 }();
+/**
+ * Generate a sharing link from a file and sharecode
+ * @param {object} options
+ * @param {object} options.client - CozyClient instance
+ * @param {object} options.file - File to share
+ * @param {string} options.sharecode - Share code
+ * @param {Function} options.getOwner - Function to get the owner of a sharing
+ * @param {Function} options.getSharingById - Function to get a sharing by ID
+ * @param {string} options.documentType - Type of the document
+ * @returns {string} - The generated web link
+ */
+
+export var generateShareLinkFromFile = function generateShareLinkFromFile(_ref3) {
+  var client = _ref3.client,
+      file = _ref3.file,
+      sharecode = _ref3.sharecode,
+      getOwner = _ref3.getOwner,
+      getSharingById = _ref3.getSharingById,
+      documentType = _ref3.documentType;
+  var cozyUrl = client.getStackClient().uri;
+
+  if (file.driveId) {
+    var _sharing$rules;
+
+    var sharing = getSharingById(file.driveId);
+    var owner = getOwner(sharing === null || sharing === void 0 || (_sharing$rules = sharing.rules) === null || _sharing$rules === void 0 || (_sharing$rules = _sharing$rules[0]) === null || _sharing$rules === void 0 || (_sharing$rules = _sharing$rules.values) === null || _sharing$rules === void 0 ? void 0 : _sharing$rules[0]);
+    cozyUrl = (owner === null || owner === void 0 ? void 0 : owner.instance) || client.getStackClient().uri;
+  }
+
+  return generateWebLink({
+    cozyUrl: cozyUrl,
+    searchParams: [['sharecode', sharecode]],
+    pathname: '/public',
+    slug: getAppSlugFromDocumentType({
+      documentType: documentType
+    }),
+    subDomainType: client.capabilities.flat_subdomains ? 'flat' : 'nested'
+  });
+};
 /**
  * Copy a string to the clipboard
  *
@@ -87,8 +124,8 @@ export var createSharingLink = /*#__PURE__*/function () {
  */
 
 export var copyToClipboard = /*#__PURE__*/function () {
-  var _ref3 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee2(value) {
-    var _ref4,
+  var _ref4 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee2(value) {
+    var _ref5,
         t,
         showAlert,
         _args2 = arguments;
@@ -96,7 +133,7 @@ export var copyToClipboard = /*#__PURE__*/function () {
     return _regeneratorRuntime.wrap(function _callee2$(_context2) {
       while (1) switch (_context2.prev = _context2.next) {
         case 0:
-          _ref4 = _args2.length > 1 && _args2[1] !== undefined ? _args2[1] : {}, t = _ref4.t, showAlert = _ref4.showAlert;
+          _ref5 = _args2.length > 1 && _args2[1] !== undefined ? _args2[1] : {}, t = _ref5.t, showAlert = _ref5.showAlert;
 
           if (value) {
             _context2.next = 3;
@@ -137,7 +174,7 @@ export var copyToClipboard = /*#__PURE__*/function () {
   }));
 
   return function copyToClipboard(_x2) {
-    return _ref3.apply(this, arguments);
+    return _ref4.apply(this, arguments);
   };
 }();
 /**
@@ -188,12 +225,12 @@ export var makeTTL = function makeTTL(selectedDate) {
  */
 
 export var createPermissions = /*#__PURE__*/function () {
-  var _ref6 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee3(_ref5) {
+  var _ref7 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee3(_ref6) {
     var file, t, ttl, password, editingRights, documentType, shareByLink, showAlert, verbs;
     return _regeneratorRuntime.wrap(function _callee3$(_context3) {
       while (1) switch (_context3.prev = _context3.next) {
         case 0:
-          file = _ref5.file, t = _ref5.t, ttl = _ref5.ttl, password = _ref5.password, editingRights = _ref5.editingRights, documentType = _ref5.documentType, shareByLink = _ref5.shareByLink, showAlert = _ref5.showAlert;
+          file = _ref6.file, t = _ref6.t, ttl = _ref6.ttl, password = _ref6.password, editingRights = _ref6.editingRights, documentType = _ref6.documentType, shareByLink = _ref6.shareByLink, showAlert = _ref6.showAlert;
           _context3.prev = 1;
           verbs = editingRights === 'readOnly' ? READ_ONLY_PERMS : WRITE_PERMS;
           return _context3.abrupt("return", shareByLink(file, {
@@ -220,7 +257,7 @@ export var createPermissions = /*#__PURE__*/function () {
   }));
 
   return function createPermissions(_x3) {
-    return _ref6.apply(this, arguments);
+    return _ref7.apply(this, arguments);
   };
 }();
 /**
@@ -239,12 +276,12 @@ export var createPermissions = /*#__PURE__*/function () {
  */
 
 export var updatePermissions = /*#__PURE__*/function () {
-  var _ref8 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee4(_ref7) {
+  var _ref9 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee4(_ref8) {
     var file, t, dateToggle, selectedDate, passwordToggle, password, editingRights, documentType, updateDocumentPermissions, showAlert, expiresAt, ensurePassword, verbs;
     return _regeneratorRuntime.wrap(function _callee4$(_context4) {
       while (1) switch (_context4.prev = _context4.next) {
         case 0:
-          file = _ref7.file, t = _ref7.t, dateToggle = _ref7.dateToggle, selectedDate = _ref7.selectedDate, passwordToggle = _ref7.passwordToggle, password = _ref7.password, editingRights = _ref7.editingRights, documentType = _ref7.documentType, updateDocumentPermissions = _ref7.updateDocumentPermissions, showAlert = _ref7.showAlert;
+          file = _ref8.file, t = _ref8.t, dateToggle = _ref8.dateToggle, selectedDate = _ref8.selectedDate, passwordToggle = _ref8.passwordToggle, password = _ref8.password, editingRights = _ref8.editingRights, documentType = _ref8.documentType, updateDocumentPermissions = _ref8.updateDocumentPermissions, showAlert = _ref8.showAlert;
           _context4.prev = 1;
           expiresAt = dateToggle ? (selectedDate === null || selectedDate === void 0 ? void 0 : selectedDate.toISOString()) || undefined : '';
           ensurePassword = passwordToggle ? password || undefined : '';
@@ -273,7 +310,7 @@ export var updatePermissions = /*#__PURE__*/function () {
   }));
 
   return function updatePermissions(_x4) {
-    return _ref8.apply(this, arguments);
+    return _ref9.apply(this, arguments);
   };
 }();
 /**
@@ -287,12 +324,12 @@ export var updatePermissions = /*#__PURE__*/function () {
  */
 
 export var revokePermissions = /*#__PURE__*/function () {
-  var _ref10 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee5(_ref9) {
+  var _ref11 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee5(_ref10) {
     var file, t, documentType, revokeSharingLink, showAlert;
     return _regeneratorRuntime.wrap(function _callee5$(_context5) {
       while (1) switch (_context5.prev = _context5.next) {
         case 0:
-          file = _ref9.file, t = _ref9.t, documentType = _ref9.documentType, revokeSharingLink = _ref9.revokeSharingLink, showAlert = _ref9.showAlert;
+          file = _ref10.file, t = _ref10.t, documentType = _ref10.documentType, revokeSharingLink = _ref10.revokeSharingLink, showAlert = _ref10.showAlert;
           _context5.prev = 1;
           return _context5.abrupt("return", revokeSharingLink(file));
 
@@ -314,6 +351,6 @@ export var revokePermissions = /*#__PURE__*/function () {
   }));
 
   return function revokePermissions(_x5) {
-    return _ref10.apply(this, arguments);
+    return _ref11.apply(this, arguments);
   };
 }();

package/dist/components/SharedDrive/DumbSharedDriveModal.js

@@ -2,7 +2,7 @@ import PropTypes from 'prop-types';
 import React from 'react';
 import { useI18n } from 'twake-i18n';
 import withLocales from '../../hoc/withLocales';
-import DumbBatchSharedFolderModal from '../SharedFolder/DumbBatchSharedFolderModal';
+import { DumbBatchSharedFolderModal } from '../SharedFolder/DumbBatchSharedFolderModal';
 export var DumbSharedDriveModal = withLocales(function (_ref) {
   var title = _ref.title,
       document = _ref.document,

package/dist/components/SharedFolder/DumbBatchSharedFolderModal.js

@@ -34,7 +34,9 @@ export var DumbBatchSharedFolderModal = withLocales(function (_ref) {
       createLabel = _ref.createLabel,
       shareLabel = _ref.shareLabel,
       saveLabel = _ref.saveLabel,
-      sharingDesc = _ref.sharingDesc;
+      sharingDesc = _ref.sharingDesc,
+      _ref$showShareByEmail = _ref.showShareByEmail,
+      showShareByEmail = _ref$showShareByEmail === void 0 ? true : _ref$showShareByEmail;
   var hasRecipients = Boolean(recipients === null || recipients === void 0 ? void 0 : recipients.length); // Determine action buttons based on modal mode
 
   var actionButtons = function () {
@@ -50,6 +52,14 @@ export var DumbBatchSharedFolderModal = withLocales(function (_ref) {
       }));
     }
 
+    if (!showShareByEmail) {
+      return /*#__PURE__*/React.createElement(ShareByLink, {
+        link: sharingLink,
+        document: document,
+        documentType: "Files"
+      });
+    }
+
     if (!showNameField) {
       return /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement(ShareByLink, {
         link: sharingLink,
@@ -104,7 +114,7 @@ export var DumbBatchSharedFolderModal = withLocales(function (_ref) {
     }), /*#__PURE__*/React.createElement(Typography, {
       variant: "h6",
       className: "u-mt-1-half u-mb-half"
-    }, addPeopleLabel), /*#__PURE__*/React.createElement(DumbShareByEmail, {
+    }, addPeopleLabel), showShareByEmail && /*#__PURE__*/React.createElement(DumbShareByEmail, {
       createContact: createContact,
       currentRecipients: currentRecipients,
       document: document,
@@ -154,6 +164,9 @@ DumbBatchSharedFolderModal.propTypes = {
   createLabel: PropTypes.string,
   shareLabel: PropTypes.string,
   saveLabel: PropTypes.string,
-  sharingDesc: PropTypes.string.isRequired
+  sharingDesc: PropTypes.string,
+  showShareByEmail: PropTypes.bool
 };
-export default DumbBatchSharedFolderModal;
+DumbBatchSharedFolderModal.defaultProps = {
+  sharingDesc: ''
+};

package/dist/components/WhoHasAccess.js

@@ -55,7 +55,6 @@ var WhoHasAccess = function WhoHasAccess(_ref2) {
     documentType: documentType,
     onRevoke: onRevoke,
     onRevokeSelf: onRevokeSelf,
-    link: link,
     permissions: permissions,
     fadeIn: linkHasBeenJustCreated
   }), /*#__PURE__*/React.createElement(OwnerRecipient, {

package/dist/helpers/shortcodes.js

@@ -0,0 +1,24 @@
+/**
+ * Get the shortcode from a permission object
+ * The shortcode can be in different places depending on how the sharing was created:
+ * - attributes.shortcodes.email
+ * - attributes.shortcodes.code
+ * - attributes.codes.email
+ * - attributes.codes.code
+ *
+ * We used to use `email` as `codes` attribute for a sharingByLink.
+ * But when we use cozy-client to create a Permission, by default
+ * the codes attribute is set to `code`. MesPapiers app is using this
+ * default behavior... So the sharing by link created by mes papiers
+ * didn't appear correctly in cozy-sharing.
+ * This is a bit ugly, we should have a better way to know if this is
+ * a sharing by link or not.
+ *
+ * @param {object} permission - The permission object
+ * @returns {string|null} The shortcode or null if not found
+ */
+export var getShortcode = function getShortcode(permission) {
+  var _permission$attribute, _permission$attribute2, _permission$attribute3, _permission$attribute4;
+
+  return (permission === null || permission === void 0 || (_permission$attribute = permission.attributes) === null || _permission$attribute === void 0 || (_permission$attribute = _permission$attribute.shortcodes) === null || _permission$attribute === void 0 ? void 0 : _permission$attribute.email) || (permission === null || permission === void 0 || (_permission$attribute2 = permission.attributes) === null || _permission$attribute2 === void 0 || (_permission$attribute2 = _permission$attribute2.shortcodes) === null || _permission$attribute2 === void 0 ? void 0 : _permission$attribute2.code) || (permission === null || permission === void 0 || (_permission$attribute3 = permission.attributes) === null || _permission$attribute3 === void 0 || (_permission$attribute3 = _permission$attribute3.codes) === null || _permission$attribute3 === void 0 ? void 0 : _permission$attribute3.email) || (permission === null || permission === void 0 || (_permission$attribute4 = permission.attributes) === null || _permission$attribute4 === void 0 || (_permission$attribute4 = _permission$attribute4.codes) === null || _permission$attribute4 === void 0 ? void 0 : _permission$attribute4.code) || null;
+};

package/dist/state.js

@@ -10,8 +10,8 @@ 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 get from 'lodash/get';
 import flag from 'cozy-flags';
+import { getShortcode } from './helpers/shortcodes';
 var RECEIVE_SHARINGS = 'RECEIVE_SHARINGS';
 var ADD_SHARING = 'ADD_SHARING';
 var UPDATE_SHARING = 'UPDATE_SHARING';
@@ -521,15 +521,8 @@ export var getSharingLink = function getSharingLink(state, docId, documentType)
   // This shouldn't have happened, but unfortunately some duplicate sharing links have been created in the past
   var perms = getDocumentPermissions(state, docId);
   if (perms.length === 0) return null;
-  var perm = perms[0]; // We used to use `email` as `codes` attribute for a sharingByLink.
-  // But when we use cozy-client to create a Permission, by default
-  // the codes attribute is set to `code`. MesPapiers app is using this
-  // default behavior... So the sharing by link created by mes papiers
-  // didn't appear correctly in cozy-sharing.
-  // This is a bit ugly, we should have a better way to know if this is
-  // a sharing by link or not.
-
-  var code = get(perm, 'attributes.shortcodes.email') || get(perm, 'attributes.shortcodes.code') || get(perm, 'attributes.codes.email') || get(perm, 'attributes.codes.code');
+  var perm = perms[0];
+  var code = getShortcode(perm);
 
   if (code) {
     return buildSharingLink(state, documentType, code);

package/locales/en.json

@@ -34,7 +34,10 @@
     "members": {
       "count": "%{smart_count} member |||| %{smart_count} members",
       "others": "and 1 other… |||| and %{smart_count} others…",
-      "otherContacts": "other contact |||| other contacts"
+      "otherContacts": "other contact |||| other contacts",
+      "error": {
+        "changePermission": "An error occurred while changing the permissions. Please try again."
+      }
     },
     "contacts": {
       "whoHasAccess": "Who has access?",

package/locales/fr.json

@@ -34,7 +34,10 @@
     "members": {
       "count": "%{smart_count} membre |||| %{smart_count} membres",
       "others": "et 1 autre… |||| et %{smart_count} autres…",
-      "otherContacts": "autre contact |||| autres contacts"
+      "otherContacts": "autre contact |||| autres contacts",
+      "error": {
+        "changePermission": "Une erreur s'est produite lors du changement des permissions. Veuillez réessayer."
+      }
     },
     "contacts": {
       "whoHasAccess": "Qui a accès ?",

package/locales/ru.json

@@ -34,7 +34,10 @@
     "members": {
       "count": "%{smart_count} участник |||| %{smart_count} участника |||| %{smart_count} участников",
       "others": "и 1 другой… |||| и %{smart_count} других…",
-      "otherContacts": "другой контакт |||| других контакта"
+      "otherContacts": "другой контакт |||| других контакта",
+      "error": {
+        "changePermission": "Произошла ошибка при изменении разрешений. Пожалуйста, попробуйте снова."
+      }
     },
     "contacts": {
       "whoHasAccess": "У кого есть доступ?",

package/locales/vi.json

@@ -34,7 +34,10 @@
     "members": {
       "count": "%{smart_count} thành viên |||| %{smart_count} thành viên",
       "others": "và 1 người khác… |||| và %{smart_count} người khác…",
-      "otherContacts": "người liên hệ khác |||| người liên hệ khác"
+      "otherContacts": "người liên hệ khác |||| người liên hệ khác",
+      "error": {
+        "changePermission": "Đã xảy ra lỗi khi thay đổi quyền. Vui lòng thử lại."
+      }
     },
     "contacts": {
       "whoHasAccess": "Ai có quyền truy cập?",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-sharing",
-  "version": "28.4.4",
+  "version": "28.6.0",
   "description": "Provides sharing login for React applications.",
   "main": "dist/index.js",
   "author": "Cozy",
@@ -54,7 +54,7 @@
     "babel-jest": "27.5.1",
     "babel-plugin-css-modules-transform": "1.6.2",
     "babel-plugin-inline-react-svg": "1.1.2",
-    "cozy-client": "^60.19.0",
+    "cozy-client": "^60.22.0",
     "cozy-device-helper": "^3.7.1",
     "cozy-intent": "^2.30.3",
     "cozy-minilog": "^3.10.0",
@@ -69,7 +69,7 @@
     "twake-i18n": "^0.3.2"
   },
   "peerDependencies": {
-    "cozy-client": ">=60.19.0",
+    "cozy-client": ">=60.22.0",
     "cozy-device-helper": ">=3.7.1",
     "cozy-intent": ">=2.29.1",
     "cozy-minilog": ">=3.9.1",
@@ -83,5 +83,5 @@
   "sideEffects": [
     "*.css"
   ],
-  "gitHead": "3b9c5cf0b6442700d6a3b9d21010089f3f9f8b32"
+  "gitHead": "64311a9d08372224b5214cbf49f7d658d93f9af9"
 }

package/src/SharingProvider.jsx

@@ -3,6 +3,7 @@ import React, { Component } from 'react'
 import { withClient } from 'cozy-client'
 import minilog from 'cozy-minilog'
 
+import { generateShareLinkFromFile } from './components/ShareRestrictionModal/helpers'
 import SharingContext from './context'
 import { fetchNextPermissions } from './fetchNextPermissions'
 import { fetchFilesPaths } from './helpers/files'
@@ -11,6 +12,7 @@ import {
   createSharingInStore,
   updateSharingInStore
 } from './helpers/sharings'
+import { getShortcode } from './helpers/shortcodes'
 import { SynchronousJobQueue } from './helpers/synchronousJobQueue'
 import { fetchApps } from './queries/queries'
 import reducer, {
@@ -67,6 +69,7 @@ export class SharingProvider extends Component {
       canReshare: docId => canReshare(this.state, docId, instanceUri),
       getOwner: docId => getOwner(this.state, docId),
       getSharingType: docId => getSharingType(this.state, docId, instanceUri),
+      getSharingById: docId => getSharingById(this.state, docId),
       getSharingForSelf: docId => getSharingForSelf(this.state, docId),
       getRecipients: docId => getRecipients(this.state, docId),
       getSharedParentPath: documentPath =>
@@ -83,6 +86,7 @@ export class SharingProvider extends Component {
       revokeGroup: this.revokeGroup,
       revokeSelf: this.revokeSelf,
       shareByLink: this.shareByLink,
+      getFederatedShareLink: this.getFederatedShareLink,
       updateDocumentPermissions: this.updateDocumentPermissions,
       revokeSharingLink: this.revokeSharingLink,
       hasLoadedAtLeastOnePage: false,
@@ -90,7 +94,8 @@ export class SharingProvider extends Component {
       revokeAllRecipients: this.revokeAllRecipients,
       refresh: this.fetchAllSharings,
       hasWriteAccess: this.hasWriteAccess,
-      renameSharedDrive: this.renameSharedDrive
+      renameSharedDrive: this.renameSharedDrive,
+      updateSharingMemberType: this.updateSharingMemberType
     }
     this.isPublic = props.isPublic ?? false
     this.realtime = null
@@ -335,12 +340,116 @@ export class SharingProvider extends Component {
     this.dispatch(revokeSelf(sharing))
   }
 
+  /**
+   * 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
+   */
+  updateSharingMemberType = async (sharingId, memberIndex, newType) => {
+    const sharing = getSharingById(this.state, sharingId)
+    if (!sharing) throw new Error('Sharing not found')
+
+    const member = sharing.attributes.members[memberIndex]
+    if (!member) throw new Error('Member not found')
+
+    // Find the contact by email (same pattern as helpers/contacts.js)
+    const matchedContacts = await this.props.client
+      .collection('io.cozy.contacts')
+      .find(
+        {
+          email: { $elemMatch: { address: member.email } },
+          _id: { $gt: null }
+        },
+        { indexedFields: ['_id'] }
+      )
+
+    if (!matchedContacts.data.length) {
+      throw new Error(`Contact not found for member ${member.email}`)
+    }
+    const contact = matchedContacts.data[0]
+
+    // Revoke (API call only, no state dispatch to avoid UI flickering)
+    await this.sharingCol.revokeRecipient(sharing, memberIndex)
+
+    // Re-add with the new permission type
+    const readOnly = newType === 'one-way'
+    try {
+      await this.addRecipients({
+        document: sharing,
+        recipients: readOnly ? [] : [contact],
+        readOnlyRecipients: readOnly ? [contact] : []
+      })
+    } catch (error) {
+      log.error(
+        `Failed to re-add member ${member.email} after revocation. The member has been revoked but could not be re-added with the new permission type.`,
+        error
+      )
+      throw error
+    }
+  }
+
   shareByLink = async (document, options) => {
-    const resp = await this.permissionCol.createSharingLink(document, options)
+    const resp = document.driveId
+      ? await this.createSharedDriveSharingLink(document, options)
+      : await this.permissionCol.createSharingLink(document, options)
     this.dispatch(addSharingLink(resp.data))
     return resp
   }
 
+  getFederatedShareLink = document => {
+    if (!document.driveId) return null
+
+    const permissions = getDocumentPermissions(this.state, document._id)
+    const perm = permissions.find(p => getShortcode(p))
+    if (!perm) return null
+
+    const code = getShortcode(perm)
+    if (!code) return null
+
+    const { client } = this.props
+    return generateShareLinkFromFile({
+      client,
+      file: document,
+      sharecode: code,
+      getOwner: this.state.getOwner,
+      getSharingById: this.state.getSharingById,
+      documentType: this.state.documentType
+    })
+  }
+
+  createSharedDriveSharingLink = async (document, options) => {
+    const { client } = this.props
+    const drivePermissionCollection = client.collection('io.cozy.permissions', {
+      driveId: document.driveId
+    })
+    const resp = await drivePermissionCollection.createSharingLink(
+      document,
+      options
+    )
+    return resp
+  }
+
+  updateSharedDrivePermissions = async (
+    document,
+    permissionDocument,
+    updatedPermissions,
+    options
+  ) => {
+    const { client } = this.props
+    const { expiresAt, password } = options
+    const drivePermissionCollection = client.collection('io.cozy.permissions', {
+      driveId: document.driveId
+    })
+    const resp = await drivePermissionCollection.add(
+      permissionDocument,
+      updatedPermissions,
+      { expiresAt, password }
+    )
+    this.dispatch(updateSharingLink(resp.data))
+    return resp
+  }
+
   /**
    * updateDocumentPermissions - Description
    *
@@ -356,19 +465,29 @@ export class SharingProvider extends Component {
     const { verbs, expiresAt, password } = options
     const permissions = getDocumentPermissions(this.state, document.id)
 
+    if (!permissions || permissions.length === 0) {
+      return []
+    }
+
     const responses = await Promise.all(
       permissions.map(async permissionDocument => {
         const updatedPermissions = permissionDocument.attributes.permissions
-        Object.keys(updatedPermissions).map(permType => {
+        Object.keys(updatedPermissions).forEach(permType => {
           updatedPermissions[permType].verbs = verbs
         })
 
-        const resp = await this.permissionCol.add(
-          permissionDocument,
-          updatedPermissions,
-          { expiresAt, password }
-        )
-        this.dispatch(updateSharingLink(resp.data))
+        const resp = document.driveId
+          ? await this.updateSharedDrivePermissions(
+              document,
+              permissionDocument,
+              updatedPermissions,
+              options
+            )
+          : await this.permissionCol.add(
+              permissionDocument,
+              updatedPermissions,
+              { expiresAt, password }
+            )
         return resp
       })
     )