cozy-sharing 26.1.1 → 26.2.0

package/CHANGELOG.md

@@ -3,6 +3,17 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [26.2.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@26.1.1...cozy-sharing@26.2.0) (2025-09-18)
+
+### Features
+
+- **sharing:** Add isSharedDrive selector ([50212d4](https://github.com/cozy/cozy-libs/commit/50212d4f88976633bb6d26830e759887ed593cc1))
+- **sharing:** Add shared drive edit modal ([112ba42](https://github.com/cozy/cozy-libs/commit/112ba429876edc9a57884b55352126a6e29786e4))
+- **sharing:** Allow shared drive modal to hide some inputs ([7a312d1](https://github.com/cozy/cozy-libs/commit/7a312d14743d1c1bfd70d15618615e75ce875274))
+- **sharing:** Display good wording when removing access in a ([d78eb48](https://github.com/cozy/cozy-libs/commit/d78eb481918bb97ee771b7e65b130854994317c4))
+- **sharing:** Properly display readonly for shared drives recipients ([cb18bd5](https://github.com/cozy/cozy-libs/commit/cb18bd506e1352875781f2585ca3d33a6aaae098))
+- **sharing:** Save shared drive title in description during creation ([52a82ad](https://github.com/cozy/cozy-libs/commit/52a82ad7e7eadfa4306fc5836f497a5ec91074f5))
+
 ## [26.1.1](https://github.com/cozy/cozy-libs/compare/cozy-sharing@26.1.0...cozy-sharing@26.1.1) (2025-09-15)
 
 ### Bug Fixes

package/dist/SharingProvider.js

@@ -39,7 +39,7 @@ import { fetchFilesPaths } from './helpers/files';
 import { getSharingObject, createSharingInStore, updateSharingInStore } from './helpers/sharings';
 import { SynchronousJobQueue } from './helpers/synchronousJobQueue';
 import { fetchApps } from './queries/queries';
-import reducer, { receiveSharings, addSharing, updateSharing, addSharingLink, updateSharingLink, revokeSharingLink, revokeRecipient, revokeGroup as revokeGroupFromState, revokeSelf, receivePaths, isOwner as _isOwner, canReshare as _canReshare, getOwner as _getOwner, getRecipients as _getRecipients, getSharingById, getExternalSharingIds, getSharingForSelf as _getSharingForSelf, getSharingType as _getSharingType, getSharingLink as _getSharingLink, getSharedDocIdsBySharings, getDocumentSharing, getDocumentPermissions as _getDocumentPermissions, hasSharedParent as _hasSharedParent, hasSharedChild as _hasSharedChild, getSharedParentPath as _getSharedParentPath } from './state';
+import reducer, { receiveSharings, addSharing, updateSharing, addSharingLink, updateSharingLink, revokeSharingLink, revokeRecipient, revokeGroup as revokeGroupFromState, revokeSelf, receivePaths, isOwner as _isOwner, isSharedDrive as _isSharedDrive, canReshare as _canReshare, getOwner as _getOwner, getRecipients as _getRecipients, getSharingById, getExternalSharingIds, getSharingForSelf as _getSharingForSelf, getSharingType as _getSharingType, getSharingLink as _getSharingLink, getSharedDocIdsBySharings, getDocumentSharing, getDocumentPermissions as _getDocumentPermissions, hasSharedParent as _hasSharedParent, hasSharedChild as _hasSharedChild, getSharedParentPath as _getSharedParentPath } from './state';
 var log = minilog('SharingProvider');
 var SHARING_DOCTYPE = 'io.cozy.sharings';
 var PERMISSION_DOCTYPE = 'io.cozy.permissions';
@@ -582,6 +582,9 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
       isOwner: function isOwner(docId) {
         return _isOwner(_this2.state, docId);
       },
+      isSharedDrive: function isSharedDrive(docId) {
+        return _isSharedDrive(_this2.state, docId);
+      },
       canReshare: function canReshare(docId) {
         return _canReshare(_this2.state, docId, _instanceUri);
       },

package/dist/components/Recipient/MemberRecipientPermissions.js

@@ -14,6 +14,7 @@ import { revokeMember } from './actions/revokeMember';
 
 var MemberRecipientPermissions = function MemberRecipientPermissions(_ref) {
   var isOwner = _ref.isOwner,
+      isSharedDrive = _ref.isSharedDrive,
       status = _ref.status,
       instance = _ref.instance,
       type = _ref.type,
@@ -87,6 +88,7 @@ var MemberRecipientPermissions = function MemberRecipientPermissions(_ref) {
     t: t,
     type: type !== null && type !== void 0 ? type : 'one-way',
     isOwner: isOwner,
+    isSharedDrive: isSharedDrive,
     handleRevocation: handleRevocation
   });
   return /*#__PURE__*/React.createElement("div", {

package/dist/components/Recipient/RecipientList.js

@@ -10,6 +10,7 @@ var RecipientList = function RecipientList(_ref) {
   var recipients = _ref.recipients,
       recipientsToBeConfirmed = _ref.recipientsToBeConfirmed,
       isOwner = _ref.isOwner,
+      isSharedDrive = _ref.isSharedDrive,
       document = _ref.document,
       documentType = _ref.documentType,
       onRevoke = _ref.onRevoke,
@@ -52,6 +53,7 @@ var RecipientList = function RecipientList(_ref) {
     return /*#__PURE__*/React.createElement(MemberRecipient, _extends({}, recipient, {
       key: recipient.index,
       isOwner: isOwner,
+      isSharedDrive: isSharedDrive,
       document: document,
       documentType: documentType,
       onRevoke: onRevoke,

package/dist/components/Recipient/actions/revokeGroup.js

@@ -10,10 +10,11 @@ import Typography from 'cozy-ui/transpiled/react/Typography';
 var revokeGroup = function revokeGroup(_ref) {
   var t = _ref.t,
       isOwner = _ref.isOwner,
+      isSharedDrive = _ref.isSharedDrive,
       handleRevocation = _ref.handleRevocation;
   var revokeType = isOwner ? 'revoke' : 'revokeSelf';
   var title = t("RevokeGroupItem.".concat(revokeType, ".title"));
-  var desc = t("RevokeGroupItem.".concat(revokeType, ".desc"));
+  var desc = isOwner && isSharedDrive ? t("RevokeGroupItem.revoke.sharedDriveDesc") : t("RevokeGroupItem.".concat(revokeType, ".desc"));
   var icon = ForbiddenIcon;
   return {
     name: 'revokeMember',

package/dist/components/Recipient/actions/revokeMember.js

@@ -10,10 +10,11 @@ import Typography from 'cozy-ui/transpiled/react/Typography';
 var revokeMember = function revokeMember(_ref) {
   var t = _ref.t,
       isOwner = _ref.isOwner,
+      isSharedDrive = _ref.isSharedDrive,
       handleRevocation = _ref.handleRevocation;
   var revokeType = isOwner ? 'revoke' : 'revokeSelf';
   var title = t("RevokeMemberItem.".concat(revokeType, ".title"));
-  var desc = t("RevokeMemberItem.".concat(revokeType, ".desc"));
+  var desc = isOwner && isSharedDrive ? t("RevokeMemberItem.revoke.sharedDriveDesc") : t("RevokeMemberItem.".concat(revokeType, ".desc"));
   var icon = TrashIcon;
   return {
     name: 'revokeMember',

package/dist/components/ShareModal/EditableSharingModal.js

@@ -24,6 +24,7 @@ export var EditableSharingModal = function EditableSharingModal(_ref) {
       hasSharedChild = _useSharingContext.hasSharedChild,
       hasSharedParent = _useSharingContext.hasSharedParent,
       isOwner = _useSharingContext.isOwner,
+      isSharedDrive = _useSharingContext.isSharedDrive,
       revoke = _useSharingContext.revoke,
       revokeSelf = _useSharingContext.revokeSelf,
       share = _useSharingContext.share;
@@ -37,6 +38,7 @@ export var EditableSharingModal = function EditableSharingModal(_ref) {
     hasSharedChild: documentPath && hasSharedChild(documentPath),
     hasSharedParent: documentPath && hasSharedParent(documentPath),
     isOwner: isOwner(document._id),
+    isSharedDrive: isSharedDrive(document._id),
     link: getSharingLink(document._id),
     onRevoke: revoke,
     onRevokeSelf: revokeSelf,

package/dist/components/ShareModal.js

@@ -3,6 +3,7 @@ import React from 'react';
 import flag from 'cozy-flags';
 import ShareDialogCozyToCozy from './ShareDialogCozyToCozy';
 import ShareDialogOnlyByLink from './ShareDialogOnlyByLink';
+import { SharedDriveEditModal } from './SharedDrive/SharedDriveEditModal';
 export var ShareModal = function ShareModal(_ref) {
   var createContact = _ref.createContact,
       document = _ref.document,
@@ -11,6 +12,7 @@ export var ShareModal = function ShareModal(_ref) {
       hasSharedChild = _ref.hasSharedChild,
       hasSharedParent = _ref.hasSharedParent,
       isOwner = _ref.isOwner,
+      isSharedDrive = _ref.isSharedDrive,
       link = _ref.link,
       onClose = _ref.onClose,
       onRevoke = _ref.onRevoke,
@@ -21,6 +23,18 @@ export var ShareModal = function ShareModal(_ref) {
       sharing = _ref.sharing,
       sharingDesc = _ref.sharingDesc,
       twoStepsConfirmationMethods = _ref.twoStepsConfirmationMethods;
+
+  if (isSharedDrive) {
+    return /*#__PURE__*/React.createElement(SharedDriveEditModal, {
+      document: document,
+      sharing: sharing,
+      recipients: recipients,
+      onShare: onShare,
+      onRevoke: onRevoke,
+      onClose: onClose
+    });
+  }
+
   var shareDialogOnlyByLink = flag('cozy.hide-sharing-cozy-to-cozy') || documentType === 'Albums';
   var showShareByEmail = !hasSharedParent && !hasSharedChild;
   var showShareByLink = documentType !== 'Organizations';

package/dist/components/SharedDrive/DumbSharedDriveModal.js

@@ -0,0 +1,86 @@
+import PropTypes from 'prop-types';
+import React from 'react';
+import Button from 'cozy-ui/transpiled/react/Buttons';
+import { FixedDialog } from 'cozy-ui/transpiled/react/CozyDialogs';
+import TextField from 'cozy-ui/transpiled/react/TextField';
+import Typography from 'cozy-ui/transpiled/react/Typography';
+import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n';
+import withLocales from '../../hoc/withLocales';
+import { default as DumbShareByEmail } from '../ShareByEmail';
+import WhoHasAccess from '../WhoHasAccess';
+export var DumbSharedDriveModal = withLocales(function (_ref) {
+  var title = _ref.title,
+      document = _ref.document,
+      sharedDriveName = _ref.sharedDriveName,
+      handleSharedDriveNameChange = _ref.handleSharedDriveNameChange,
+      createContact = _ref.createContact,
+      recipients = _ref.recipients,
+      onRevoke = _ref.onRevoke,
+      onSetType = _ref.onSetType,
+      onCreate = _ref.onCreate,
+      onClose = _ref.onClose,
+      onShare = _ref.onShare;
+
+  var _useI18n = useI18n(),
+      t = _useI18n.t;
+
+  return /*#__PURE__*/React.createElement(FixedDialog, {
+    open: true,
+    disableGutters: true,
+    onClose: onClose,
+    title: title,
+    content: /*#__PURE__*/React.createElement("div", null, /*#__PURE__*/React.createElement("div", {
+      className: "u-ph-2"
+    }, handleSharedDriveNameChange && /*#__PURE__*/React.createElement(TextField, {
+      required: true,
+      label: t('SharedDrive.sharedDriveModal.nameLabel'),
+      variant: "outlined",
+      size: "small",
+      className: "u-w-100 u-mt-1-half",
+      value: sharedDriveName,
+      onChange: handleSharedDriveNameChange
+    }), /*#__PURE__*/React.createElement(Typography, {
+      variant: "h6",
+      className: "u-mt-1-half u-mb-half"
+    }, t('SharedDrive.sharedDriveModal.addPeople')), /*#__PURE__*/React.createElement(DumbShareByEmail, {
+      createContact: createContact,
+      currentRecipients: [],
+      document: document,
+      documentType: "Files",
+      onShare: onShare,
+      submitLabel: t('SharedDrive.sharedDriveModal.add'),
+      showNotifications: false
+    })), /*#__PURE__*/React.createElement(WhoHasAccess, {
+      isOwner: true,
+      isSharedDrive: true,
+      recipients: recipients,
+      document: document,
+      documentType: "Files",
+      className: "u-w-100",
+      onRevoke: onRevoke,
+      onSetType: onSetType
+    })),
+    actions: /*#__PURE__*/React.createElement(React.Fragment, null, onCreate && /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement(Button, {
+      variant: "secondary",
+      label: t('SharedDrive.sharedDriveModal.cancel'),
+      onClick: onClose
+    }), /*#__PURE__*/React.createElement(Button, {
+      variant: "primary",
+      label: t('SharedDrive.sharedDriveModal.create'),
+      onClick: onCreate
+    })))
+  });
+});
+DumbSharedDriveModal.propTypes = {
+  title: PropTypes.string,
+  document: PropTypes.object,
+  sharedDriveName: PropTypes.string,
+  handleSharedDriveNameChange: PropTypes.func.isRequired,
+  createContact: PropTypes.func.isRequired,
+  recipients: PropTypes.array,
+  onRevoke: PropTypes.func.isRequired,
+  onSetType: PropTypes.func.isRequired,
+  onCreate: PropTypes.func.isRequired,
+  onClose: PropTypes.func.isRequired,
+  onShare: PropTypes.func.isRequired
+};

package/dist/components/SharedDrive/SharedDriveEditModal.js

@@ -0,0 +1,38 @@
+import PropTypes from 'prop-types';
+import React from 'react';
+import { useClient } from 'cozy-client';
+import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n';
+import { DumbSharedDriveModal } from './DumbSharedDriveModal';
+import withLocales from '../../hoc/withLocales';
+import { Contact } from '../../models';
+export var SharedDriveEditModal = withLocales(function (_ref) {
+  var document = _ref.document,
+      sharing = _ref.sharing,
+      recipients = _ref.recipients,
+      onShare = _ref.onShare,
+      onRevoke = _ref.onRevoke,
+      onClose = _ref.onClose;
+  var client = useClient();
+
+  var _useI18n = useI18n(),
+      t = _useI18n.t;
+
+  var createContact = function createContact(contact) {
+    return client.create(Contact.doctype, contact);
+  };
+
+  return /*#__PURE__*/React.createElement(DumbSharedDriveModal, {
+    title: t('Files.share.title', {
+      name: sharing === null || sharing === void 0 ? void 0 : sharing.description
+    }),
+    document: document,
+    createContact: createContact,
+    recipients: recipients,
+    onRevoke: onRevoke,
+    onClose: onClose,
+    onShare: onShare
+  });
+});
+SharedDriveEditModal.propTypes = {
+  onClose: PropTypes.func.isRequired
+};

package/dist/components/SharedDrive/SharedDriveModal.js

@@ -5,18 +5,13 @@ import _regeneratorRuntime from "@babel/runtime/regenerator";
 import PropTypes from 'prop-types';
 import React, { useState } from 'react';
 import { useClient } from 'cozy-client';
-import Button from 'cozy-ui/transpiled/react/Buttons';
-import { FixedDialog } from 'cozy-ui/transpiled/react/CozyDialogs';
-import TextField from 'cozy-ui/transpiled/react/TextField';
-import Typography from 'cozy-ui/transpiled/react/Typography';
 import { useAlert } from 'cozy-ui/transpiled/react/providers/Alert';
 import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n';
+import { DumbSharedDriveModal } from './DumbSharedDriveModal';
 import { mergeAndDeduplicateRecipients, formatRecipients } from './helpers';
 import withLocales from '../../hoc/withLocales';
 import { useSharingContext } from '../../hooks/useSharingContext';
 import { Contact } from '../../models';
-import { default as DumbShareByEmail } from '../ShareByEmail';
-import WhoHasAccess from '../WhoHasAccess';
 export var SharedDriveModal = withLocales(function (_ref) {
   var onClose = _ref.onClose;
   var client = useClient();
@@ -78,6 +73,7 @@ export var SharedDriveModal = withLocales(function (_ref) {
             sharedDriveFolder = _yield$client$create2.data;
             _context.next = 9;
             return share({
+              description: sharedDriveName,
               document: sharedDriveFolder,
               recipients: sharedDriveRecipients.recipients,
               readOnlyRecipients: sharedDriveRecipients.readOnlyRecipients,
@@ -159,53 +155,22 @@ export var SharedDriveModal = withLocales(function (_ref) {
     });
   };
 
+  var createContact = function createContact(contact) {
+    return client.create(Contact.doctype, contact);
+  };
+
   var recipients = formatRecipients(sharedDriveRecipients);
-  return /*#__PURE__*/React.createElement(FixedDialog, {
-    open: true,
-    disableGutters: true,
-    onClose: onClose,
+  return /*#__PURE__*/React.createElement(DumbSharedDriveModal, {
     title: t('SharedDrive.sharedDriveModal.title'),
-    content: /*#__PURE__*/React.createElement("div", null, /*#__PURE__*/React.createElement("div", {
-      className: "u-ph-2"
-    }, /*#__PURE__*/React.createElement(TextField, {
-      required: true,
-      label: t('SharedDrive.sharedDriveModal.nameLabel'),
-      variant: "outlined",
-      size: "small",
-      className: "u-w-100 u-mt-1-half",
-      value: sharedDriveName,
-      onChange: handleSharedDriveNameChange
-    }), /*#__PURE__*/React.createElement(Typography, {
-      variant: "h6",
-      className: "u-mt-1-half u-mb-half"
-    }, t('SharedDrive.sharedDriveModal.addPeople')), /*#__PURE__*/React.createElement(DumbShareByEmail, {
-      createContact: function createContact(contact) {
-        return client.create(Contact.doctype, contact);
-      },
-      currentRecipients: [],
-      document: document,
-      documentType: "Files",
-      onShare: onShare,
-      submitLabel: t('SharedDrive.sharedDriveModal.add'),
-      showNotifications: false
-    })), /*#__PURE__*/React.createElement(WhoHasAccess, {
-      isOwner: true,
-      recipients: recipients,
-      document: document,
-      documentType: "Files",
-      className: "u-w-100",
-      onRevoke: onRevoke,
-      onSetType: onSetType
-    })),
-    actions: /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement(Button, {
-      variant: "secondary",
-      label: t('SharedDrive.sharedDriveModal.cancel'),
-      onClick: onClose
-    }), /*#__PURE__*/React.createElement(Button, {
-      variant: "primary",
-      label: t('SharedDrive.sharedDriveModal.create'),
-      onClick: onCreate
-    }))
+    sharedDriveName: sharedDriveName,
+    handleSharedDriveNameChange: handleSharedDriveNameChange,
+    createContact: createContact,
+    recipients: recipients,
+    onRevoke: onRevoke,
+    onSetType: onSetType,
+    onCreate: onCreate,
+    onClose: onClose,
+    onShare: onShare
   });
 });
 SharedDriveModal.propTypes = {

package/dist/components/WhoHasAccess.js

@@ -30,6 +30,8 @@ var RecipientWaitingForConfirmationAlert = function RecipientWaitingForConfirmat
 var WhoHasAccess = function WhoHasAccess(_ref2) {
   var _ref2$isOwner = _ref2.isOwner,
       isOwner = _ref2$isOwner === void 0 ? false : _ref2$isOwner,
+      _ref2$isSharedDrive = _ref2.isSharedDrive,
+      isSharedDrive = _ref2$isSharedDrive === void 0 ? false : _ref2$isSharedDrive,
       recipients = _ref2.recipients,
       _ref2$recipientsToBeC = _ref2.recipientsToBeConfirmed,
       recipientsToBeConfirmed = _ref2$recipientsToBeC === void 0 ? [] : _ref2$recipientsToBeC,
@@ -62,6 +64,7 @@ var WhoHasAccess = function WhoHasAccess(_ref2) {
     recipients: recipients,
     recipientsToBeConfirmed: recipientsToBeConfirmed,
     isOwner: isOwner,
+    isSharedDrive: isSharedDrive,
     document: document,
     documentType: documentType,
     onRevoke: onRevoke,

package/dist/state.js

@@ -397,6 +397,13 @@ export var isOwner = function isOwner(state, docId) {
 
   return true;
 };
+export var isSharedDrive = function isSharedDrive(state, docId) {
+  if (state.byDocId[docId] && state.byDocId[docId].sharings.length !== 0) {
+    return getSharingById(state, state.byDocId[docId].sharings[0]).attributes.drive === true;
+  }
+
+  return false;
+};
 export var canReshare = function canReshare(state, docId, instanceUri) {
   var sharing = getDocumentSharing(state, docId);
   var me = sharing.attributes.members.find(matchingInstanceName(instanceUri));
@@ -417,13 +424,30 @@ export var getRecipients = function getRecipients(state, docId) {
   return getRecipientsWithoutGroups(sharings, docId);
 };
 
+var getRecipientType = function getRecipientType(sharing, member, documentType) {
+  // If member is read_only, for sure it's one-way
+  if (member.read_only) {
+    return 'one-way';
+  } // Else if it is a shared drive, for sure it's two-way
+
+
+  if (sharing.drive) {
+    return 'two-way';
+  } // Else, we keep the legacy way of determining the type
+  // because we need to take some time to understand if we can
+  // just return 'two-way' or not
+
+
+  return documentType;
+};
+
 var getRecipientsWithoutGroups = function getRecipientsWithoutGroups(sharings, docId) {
   var recipients = sharings.map(function (sharing) {
     var type = getDocumentSharingType(sharing, docId);
     return sharing.attributes.members.map(function (m, idx) {
       return _objectSpread(_objectSpread({}, m), {}, {
         index: "sharing-".concat(sharing.id, "-member-").concat(idx),
-        type: m.read_only ? 'one-way' : type,
+        type: getRecipientType(sharing, m, type),
         sharingId: sharing.id,
         memberIndex: idx,
         avatarPath: "/sharings/".concat(sharing.id, "/recipients/").concat(idx, "/avatar")
@@ -452,7 +476,7 @@ export var getRecipientsWithGroups = function getRecipientsWithGroups(sharings,
     var sharingId = sharing.id;
     var members = sharing.attributes.members.map(function (m, idx) {
       return _objectSpread(_objectSpread({}, m), {}, {
-        type: m.read_only ? 'one-way' : type,
+        type: getRecipientType(sharing, m, type),
         sharingId: sharingId,
         index: "sharing-".concat(sharing.id, "-member-").concat(idx),
         memberIndex: idx,

package/locales/en.json

@@ -380,7 +380,8 @@
   "RevokeMemberItem": {
     "revoke": {
       "title": "Remove from sharing",
-      "desc": "This contact will keep a copy but the changes won't be synchrnoized anymore."
+      "desc": "This contact will keep a copy but the changes won't be synchrnoized anymore.",
+      "sharedDriveDesc": "Your contact will no longer be able to access the data."
     },
     "revokeSelf": {
       "title": "Remove me from sharing",
@@ -399,7 +400,8 @@
   "RevokeGroupItem": {
     "revoke": {
       "title": "Remove group from sharing",
-      "desc": "Group members will keep a copy but the changes won't be synchrnoized anymore."
+      "desc": "Group members will keep a copy but the changes won't be synchrnoized anymore.",
+      "sharedDriveDesc": "Group members will no longer be able to access the data."
     },
     "revokeSelf": {
       "title": "Remove me from sharing",

package/locales/fr.json

@@ -378,7 +378,8 @@
   "RevokeMemberItem": {
     "revoke": {
       "title": "Arrêter le partage",
-      "desc": "Votre contact conservera une copie mais vos changements ne seront plus synchronisés."
+      "desc": "Votre contact conservera une copie mais vos changements ne seront plus synchronisés.",
+      "sharedDriveDesc": "Votre contact ne pourra plus accéder aux données."
     },
     "revokeSelf": {
       "title": "Arrêter le partage",
@@ -397,7 +398,8 @@
   "RevokeGroupItem": {
     "revoke": {
       "title": "Arrêter le partage pour le groupe",
-      "desc": "Les membres du groupe conserveront une copie mais vos changements ne seront plus synchronisés"
+      "desc": "Les membres du groupe conserveront une copie mais vos changements ne seront plus synchronisés",
+      "sharedDriveDesc": "Les membres du groupe ne pourront plus accéder aux données."
     },
     "revokeSelf": {
       "title": "Quitter le partage",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-sharing",
-  "version": "26.1.1",
+  "version": "26.2.0",
   "description": "Provides sharing login for React applications.",
   "main": "dist/index.js",
   "author": "Cozy",
@@ -79,5 +79,5 @@
   "sideEffects": [
     "*.css"
   ],
-  "gitHead": "4fb4b75570ed5a36823d085ce27151a98d8aec1e"
+  "gitHead": "61c2e410b247be9903d2d68076fd8be5f90cba0e"
 }

package/src/SharingProvider.jsx

@@ -25,6 +25,7 @@ import reducer, {
   revokeSelf,
   receivePaths,
   isOwner,
+  isSharedDrive,
   canReshare,
   getOwner,
   getRecipients,
@@ -58,6 +59,7 @@ export class SharingProvider extends Component {
       sharedFolderPaths: [],
       documentType,
       isOwner: docId => isOwner(this.state, docId),
+      isSharedDrive: docId => isSharedDrive(this.state, docId),
       canReshare: docId => canReshare(this.state, docId, instanceUri),
       getOwner: docId => getOwner(this.state, docId),
       getSharingType: docId => getSharingType(this.state, docId, instanceUri),

package/src/components/Recipient/MemberRecipientPermissions.jsx

@@ -16,6 +16,7 @@ import { revokeMember } from './actions/revokeMember'
 
 const MemberRecipientPermissions = ({
   isOwner,
+  isSharedDrive,
   status,
   instance,
   type,
@@ -58,6 +59,7 @@ const MemberRecipientPermissions = ({
     t,
     type: type ?? 'one-way',
     isOwner,
+    isSharedDrive,
     handleRevocation
   })
 

package/src/components/Recipient/RecipientList.jsx

@@ -10,6 +10,7 @@ const RecipientList = ({
   recipients,
   recipientsToBeConfirmed,
   isOwner,
+  isSharedDrive,
   document,
   documentType,
   onRevoke,
@@ -65,6 +66,7 @@ const RecipientList = ({
         {...recipient}
         key={recipient.index}
         isOwner={isOwner}
+        isSharedDrive={isSharedDrive}
         document={document}
         documentType={documentType}
         onRevoke={onRevoke}

package/src/components/Recipient/actions/revokeGroup.js

@@ -7,10 +7,13 @@ import ListItemIcon from 'cozy-ui/transpiled/react/ListItemIcon'
 import ListItemText from 'cozy-ui/transpiled/react/ListItemText'
 import Typography from 'cozy-ui/transpiled/react/Typography'
 
-const revokeGroup = ({ t, isOwner, handleRevocation }) => {
+const revokeGroup = ({ t, isOwner, isSharedDrive, handleRevocation }) => {
   const revokeType = isOwner ? 'revoke' : 'revokeSelf'
   const title = t(`RevokeGroupItem.${revokeType}.title`)
-  const desc = t(`RevokeGroupItem.${revokeType}.desc`)
+  const desc =
+    isOwner && isSharedDrive
+      ? t(`RevokeGroupItem.revoke.sharedDriveDesc`)
+      : t(`RevokeGroupItem.${revokeType}.desc`)
 
   const icon = ForbiddenIcon
 

package/src/components/Recipient/actions/revokeMember.js

@@ -7,10 +7,13 @@ import ListItemIcon from 'cozy-ui/transpiled/react/ListItemIcon'
 import ListItemText from 'cozy-ui/transpiled/react/ListItemText'
 import Typography from 'cozy-ui/transpiled/react/Typography'
 
-const revokeMember = ({ t, isOwner, handleRevocation }) => {
+const revokeMember = ({ t, isOwner, isSharedDrive, handleRevocation }) => {
   const revokeType = isOwner ? 'revoke' : 'revokeSelf'
   const title = t(`RevokeMemberItem.${revokeType}.title`)
-  const desc = t(`RevokeMemberItem.${revokeType}.desc`)
+  const desc =
+    isOwner && isSharedDrive
+      ? t(`RevokeMemberItem.revoke.sharedDriveDesc`)
+      : t(`RevokeMemberItem.${revokeType}.desc`)
 
   const icon = TrashIcon
 

package/src/components/ShareModal/EditableSharingModal.jsx

@@ -21,6 +21,7 @@ export const EditableSharingModal = ({ document, ...rest }) => {
     hasSharedChild,
     hasSharedParent,
     isOwner,
+    isSharedDrive,
     revoke,
     revokeSelf,
     share
@@ -34,6 +35,7 @@ export const EditableSharingModal = ({ document, ...rest }) => {
       hasSharedChild={documentPath && hasSharedChild(documentPath)}
       hasSharedParent={documentPath && hasSharedParent(documentPath)}
       isOwner={isOwner(document._id)}
+      isSharedDrive={isSharedDrive(document._id)}
       link={getSharingLink(document._id)}
       onRevoke={revoke}
       onRevokeSelf={revokeSelf}

package/src/components/ShareModal.jsx

@@ -5,6 +5,7 @@ import flag from 'cozy-flags'
 
 import ShareDialogCozyToCozy from './ShareDialogCozyToCozy'
 import ShareDialogOnlyByLink from './ShareDialogOnlyByLink'
+import { SharedDriveEditModal } from './SharedDrive/SharedDriveEditModal'
 
 export const ShareModal = ({
   createContact,
@@ -13,6 +14,7 @@ export const ShareModal = ({
   hasSharedChild,
   hasSharedParent,
   isOwner,
+  isSharedDrive,
   link,
   onClose,
   onRevoke,
@@ -24,6 +26,19 @@ export const ShareModal = ({
   sharingDesc,
   twoStepsConfirmationMethods
 }) => {
+  if (isSharedDrive) {
+    return (
+      <SharedDriveEditModal
+        document={document}
+        sharing={sharing}
+        recipients={recipients}
+        onShare={onShare}
+        onRevoke={onRevoke}
+        onClose={onClose}
+      />
+    )
+  }
+
   const shareDialogOnlyByLink =
     flag('cozy.hide-sharing-cozy-to-cozy') || documentType === 'Albums'
 

package/src/components/SharedDrive/DumbSharedDriveModal.jsx

@@ -0,0 +1,110 @@
+import PropTypes from 'prop-types'
+import React from 'react'
+
+import Button from 'cozy-ui/transpiled/react/Buttons'
+import { FixedDialog } from 'cozy-ui/transpiled/react/CozyDialogs'
+import TextField from 'cozy-ui/transpiled/react/TextField'
+import Typography from 'cozy-ui/transpiled/react/Typography'
+import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n'
+
+import withLocales from '../../hoc/withLocales'
+import { default as DumbShareByEmail } from '../ShareByEmail'
+import WhoHasAccess from '../WhoHasAccess'
+
+export const DumbSharedDriveModal = withLocales(
+  ({
+    title,
+    document,
+    sharedDriveName,
+    handleSharedDriveNameChange,
+    createContact,
+    recipients,
+    onRevoke,
+    onSetType,
+    onCreate,
+    onClose,
+    onShare
+  }) => {
+    const { t } = useI18n()
+
+    return (
+      <FixedDialog
+        open
+        disableGutters
+        onClose={onClose}
+        title={title}
+        content={
+          <div>
+            <div className="u-ph-2">
+              {handleSharedDriveNameChange && (
+                <TextField
+                  required
+                  label={t('SharedDrive.sharedDriveModal.nameLabel')}
+                  variant="outlined"
+                  size="small"
+                  className="u-w-100 u-mt-1-half"
+                  value={sharedDriveName}
+                  onChange={handleSharedDriveNameChange}
+                />
+              )}
+              <Typography variant="h6" className="u-mt-1-half u-mb-half">
+                {t('SharedDrive.sharedDriveModal.addPeople')}
+              </Typography>
+              <DumbShareByEmail
+                createContact={createContact}
+                currentRecipients={[]}
+                document={document}
+                documentType="Files"
+                onShare={onShare}
+                submitLabel={t('SharedDrive.sharedDriveModal.add')}
+                showNotifications={false}
+              />
+            </div>
+            <WhoHasAccess
+              isOwner
+              isSharedDrive
+              recipients={recipients}
+              document={document}
+              documentType="Files"
+              className="u-w-100"
+              onRevoke={onRevoke}
+              onSetType={onSetType}
+            />
+          </div>
+        }
+        actions={
+          <>
+            {onCreate && (
+              <>
+                <Button
+                  variant="secondary"
+                  label={t('SharedDrive.sharedDriveModal.cancel')}
+                  onClick={onClose}
+                />
+                <Button
+                  variant="primary"
+                  label={t('SharedDrive.sharedDriveModal.create')}
+                  onClick={onCreate}
+                />
+              </>
+            )}
+          </>
+        }
+      />
+    )
+  }
+)
+
+DumbSharedDriveModal.propTypes = {
+  title: PropTypes.string,
+  document: PropTypes.object,
+  sharedDriveName: PropTypes.string,
+  handleSharedDriveNameChange: PropTypes.func.isRequired,
+  createContact: PropTypes.func.isRequired,
+  recipients: PropTypes.array,
+  onRevoke: PropTypes.func.isRequired,
+  onSetType: PropTypes.func.isRequired,
+  onCreate: PropTypes.func.isRequired,
+  onClose: PropTypes.func.isRequired,
+  onShare: PropTypes.func.isRequired
+}

package/src/components/SharedDrive/SharedDriveEditModal.jsx

@@ -0,0 +1,34 @@
+import PropTypes from 'prop-types'
+import React from 'react'
+
+import { useClient } from 'cozy-client'
+import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n'
+
+import { DumbSharedDriveModal } from './DumbSharedDriveModal'
+import withLocales from '../../hoc/withLocales'
+import { Contact } from '../../models'
+
+export const SharedDriveEditModal = withLocales(
+  ({ document, sharing, recipients, onShare, onRevoke, onClose }) => {
+    const client = useClient()
+    const { t } = useI18n()
+
+    const createContact = contact => client.create(Contact.doctype, contact)
+
+    return (
+      <DumbSharedDriveModal
+        title={t('Files.share.title', { name: sharing?.description })}
+        document={document}
+        createContact={createContact}
+        recipients={recipients}
+        onRevoke={onRevoke}
+        onClose={onClose}
+        onShare={onShare}
+      />
+    )
+  }
+)
+
+SharedDriveEditModal.propTypes = {
+  onClose: PropTypes.func.isRequired
+}

package/src/components/SharedDrive/SharedDriveModal.jsx

@@ -2,19 +2,14 @@ import PropTypes from 'prop-types'
 import React, { useState } from 'react'
 
 import { useClient } from 'cozy-client'
-import Button from 'cozy-ui/transpiled/react/Buttons'
-import { FixedDialog } from 'cozy-ui/transpiled/react/CozyDialogs'
-import TextField from 'cozy-ui/transpiled/react/TextField'
-import Typography from 'cozy-ui/transpiled/react/Typography'
 import { useAlert } from 'cozy-ui/transpiled/react/providers/Alert'
 import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n'
 
+import { DumbSharedDriveModal } from './DumbSharedDriveModal'
 import { mergeAndDeduplicateRecipients, formatRecipients } from './helpers'
 import withLocales from '../../hoc/withLocales'
 import { useSharingContext } from '../../hooks/useSharingContext'
 import { Contact } from '../../models'
-import { default as DumbShareByEmail } from '../ShareByEmail'
-import WhoHasAccess from '../WhoHasAccess'
 
 export const SharedDriveModal = withLocales(({ onClose }) => {
   const client = useClient()
@@ -57,6 +52,7 @@ export const SharedDriveModal = withLocales(({ onClose }) => {
       })
 
       await share({
+        description: sharedDriveName,
         document: sharedDriveFolder,
         recipients: sharedDriveRecipients.recipients,
         readOnlyRecipients: sharedDriveRecipients.readOnlyRecipients,
@@ -114,64 +110,22 @@ export const SharedDriveModal = withLocales(({ onClose }) => {
     })
   }
 
+  const createContact = contact => client.create(Contact.doctype, contact)
+
   const recipients = formatRecipients(sharedDriveRecipients)
 
   return (
-    <FixedDialog
-      open
-      disableGutters
-      onClose={onClose}
+    <DumbSharedDriveModal
       title={t('SharedDrive.sharedDriveModal.title')}
-      content={
-        <div>
-          <div className="u-ph-2">
-            <TextField
-              required
-              label={t('SharedDrive.sharedDriveModal.nameLabel')}
-              variant="outlined"
-              size="small"
-              className="u-w-100 u-mt-1-half"
-              value={sharedDriveName}
-              onChange={handleSharedDriveNameChange}
-            />
-            <Typography variant="h6" className="u-mt-1-half u-mb-half">
-              {t('SharedDrive.sharedDriveModal.addPeople')}
-            </Typography>
-            <DumbShareByEmail
-              createContact={contact => client.create(Contact.doctype, contact)}
-              currentRecipients={[]}
-              document={document}
-              documentType="Files"
-              onShare={onShare}
-              submitLabel={t('SharedDrive.sharedDriveModal.add')}
-              showNotifications={false}
-            />
-          </div>
-          <WhoHasAccess
-            isOwner
-            recipients={recipients}
-            document={document}
-            documentType="Files"
-            className="u-w-100"
-            onRevoke={onRevoke}
-            onSetType={onSetType}
-          />
-        </div>
-      }
-      actions={
-        <>
-          <Button
-            variant="secondary"
-            label={t('SharedDrive.sharedDriveModal.cancel')}
-            onClick={onClose}
-          />
-          <Button
-            variant="primary"
-            label={t('SharedDrive.sharedDriveModal.create')}
-            onClick={onCreate}
-          />
-        </>
-      }
+      sharedDriveName={sharedDriveName}
+      handleSharedDriveNameChange={handleSharedDriveNameChange}
+      createContact={createContact}
+      recipients={recipients}
+      onRevoke={onRevoke}
+      onSetType={onSetType}
+      onCreate={onCreate}
+      onClose={onClose}
+      onShare={onShare}
     />
   )
 })

package/src/components/WhoHasAccess.jsx

@@ -28,6 +28,7 @@ const RecipientWaitingForConfirmationAlert = ({ recipientsToBeConfirmed }) => {
 
 const WhoHasAccess = ({
   isOwner = false,
+  isSharedDrive = false,
   recipients,
   recipientsToBeConfirmed = [],
   document,
@@ -67,6 +68,7 @@ const WhoHasAccess = ({
           recipients={recipients}
           recipientsToBeConfirmed={recipientsToBeConfirmed}
           isOwner={isOwner}
+          isSharedDrive={isSharedDrive}
           document={document}
           documentType={documentType}
           onRevoke={onRevoke}

package/src/state.js

@@ -300,6 +300,16 @@ export const isOwner = (state, docId) => {
   return true
 }
 
+export const isSharedDrive = (state, docId) => {
+  if (state.byDocId[docId] && state.byDocId[docId].sharings.length !== 0) {
+    return (
+      getSharingById(state, state.byDocId[docId].sharings[0]).attributes
+        .drive === true
+    )
+  }
+  return false
+}
+
 export const canReshare = (state, docId, instanceUri) => {
   const sharing = getDocumentSharing(state, docId)
   const me = sharing.attributes.members.find(matchingInstanceName(instanceUri))
@@ -318,6 +328,23 @@ export const getRecipients = (state, docId) => {
   return getRecipientsWithoutGroups(sharings, docId)
 }
 
+const getRecipientType = (sharing, member, documentType) => {
+  // If member is read_only, for sure it's one-way
+  if (member.read_only) {
+    return 'one-way'
+  }
+
+  // Else if it is a shared drive, for sure it's two-way
+  if (sharing.drive) {
+    return 'two-way'
+  }
+
+  // Else, we keep the legacy way of determining the type
+  // because we need to take some time to understand if we can
+  // just return 'two-way' or not
+  return documentType
+}
+
 const getRecipientsWithoutGroups = (sharings, docId) => {
   const recipients = sharings
     .map(sharing => {
@@ -325,7 +352,7 @@ const getRecipientsWithoutGroups = (sharings, docId) => {
       return sharing.attributes.members.map((m, idx) => ({
         ...m,
         index: `sharing-${sharing.id}-member-${idx}`,
-        type: m.read_only ? 'one-way' : type,
+        type: getRecipientType(sharing, m, type),
         sharingId: sharing.id,
         memberIndex: idx,
         avatarPath: `/sharings/${sharing.id}/recipients/${idx}/avatar`
@@ -346,7 +373,7 @@ export const getRecipientsWithGroups = (sharings, docId) => {
 
     const members = sharing.attributes.members.map((m, idx) => ({
       ...m,
-      type: m.read_only ? 'one-way' : type,
+      type: getRecipientType(sharing, m, type),
       sharingId,
       index: `sharing-${sharing.id}-member-${idx}`,
       memberIndex: idx,