cozy-sharing 31.0.2 → 31.2.0

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

@@ -5,6 +5,8 @@ import { fireEvent, render, waitFor } from '@testing-library/react';
 import React from 'react';
 import { createMockClient } from 'cozy-client';
 import { FederatedFolderModal } from './FederatedFolderModal';
+import { getOrCreateFromArray } from '../../helpers/contacts';
+import { usePendingRecipients } from '../../hooks/usePendingRecipients';
 import AppLike from '../SharingBanner/test/AppLike';
 var mockShare = jest.fn();
 var mockRevoke = jest.fn();
@@ -39,19 +41,27 @@ jest.mock('../../hoc/withLocales', function () {
     return Component;
   };
 });
+jest.mock('../../helpers/contacts', function () {
+  return {
+    getOrCreateFromArray: jest.fn()
+  };
+});
+jest.mock('../../hooks/usePendingRecipients', function () {
+  return {
+    usePendingRecipients: jest.fn()
+  };
+});
 jest.mock('./DumbFederatedFolderModal', function () {
   return {
     DumbFederatedFolderModal: function DumbFederatedFolderModal(_ref) {
       var title = _ref.title,
-          createContact = _ref.createContact,
           recipients = _ref.recipients,
-          readOnlyRecipients = _ref.readOnlyRecipients,
           onRevoke = _ref.onRevoke,
-          onSetType = _ref.onSetType,
           onSend = _ref.onSend,
           onClose = _ref.onClose,
-          onShare = _ref.onShare,
-          sharingLink = _ref.sharingLink;
+          sharingLink = _ref.sharingLink,
+          pendingRecipients = _ref.pendingRecipients,
+          selectedOption = _ref.selectedOption;
       return /*#__PURE__*/React.createElement("div", {
         "data-testid": "dumb-modal"
       }, /*#__PURE__*/React.createElement("span", {
@@ -61,75 +71,23 @@ jest.mock('./DumbFederatedFolderModal', function () {
       }, sharingLink), /*#__PURE__*/React.createElement("span", {
         "data-testid": "recipients-count"
       }, recipients.length), /*#__PURE__*/React.createElement("span", {
-        "data-testid": "readOnly-recipients-count"
-      }, readOnlyRecipients ? readOnlyRecipients.length : 0), /*#__PURE__*/React.createElement("button", {
-        "data-testid": "btn-share",
-        onClick: function onClick() {
-          return onShare({
-            recipients: [{
-              _id: 'r1',
-              id: 'r1',
-              displayName: 'Alice'
-            }],
-            readOnlyRecipients: [{
-              _id: 'r2',
-              id: 'r2',
-              displayName: 'Bob'
-            }]
-          });
-        }
-      }, "Share"), /*#__PURE__*/React.createElement("button", {
-        "data-testid": "btn-share-other",
-        onClick: function onClick() {
-          return onShare({
-            recipients: [{
-              _id: 'r1',
-              id: 'r1',
-              displayName: 'Alice'
-            }, {
-              _id: 'r3',
-              id: 'r3',
-              displayName: 'Charlie'
-            }],
-            readOnlyRecipients: []
-          });
-        }
-      }, "Share Other"), /*#__PURE__*/React.createElement("button", {
+        "data-testid": "pending-recipients-count"
+      }, pendingRecipients.length), /*#__PURE__*/React.createElement("span", {
+        "data-testid": "selected-option"
+      }, selectedOption), /*#__PURE__*/React.createElement("button", {
         "data-testid": "btn-send",
         onClick: onSend
       }, "Send"), /*#__PURE__*/React.createElement("button", {
-        "data-testid": "btn-set-type-two-way",
-        onClick: function onClick() {
-          return onSetType('virtual-shared-drive-sharing-r2', 'two-way');
-        }
-      }, "Set Two-Way"), /*#__PURE__*/React.createElement("button", {
-        "data-testid": "btn-set-type-one-way",
-        onClick: function onClick() {
-          return onSetType('virtual-shared-drive-sharing-r1', 'one-way');
-        }
-      }, "Set One-Way"), /*#__PURE__*/React.createElement("button", {
         "data-testid": "btn-revoke",
-        onClick: function onClick() {
-          return onRevoke('virtual-shared-drive-sharing-r1');
-        }
-      }, "Revoke"), /*#__PURE__*/React.createElement("button", {
-        "data-testid": "btn-revoke-existing",
         onClick: function onClick() {
           return onRevoke({
             _id: 'folder-123'
           }, 'abc', 1);
         }
-      }, "Revoke Existing"), /*#__PURE__*/React.createElement("button", {
+      }, "Revoke"), /*#__PURE__*/React.createElement("button", {
         "data-testid": "btn-close",
         onClick: onClose
-      }, "Close"), /*#__PURE__*/React.createElement("button", {
-        "data-testid": "btn-create-contact",
-        onClick: function onClick() {
-          return createContact({
-            email: 'test@example.com'
-          });
-        }
-      }, "Create Contact"));
+      }, "Close"));
     }
   };
 });
@@ -182,6 +140,13 @@ describe('FederatedFolderModal', function () {
     mockGetRecipients.mockReturnValue([]);
     client = createTestClient();
     sharingContextValue = createSharingContextValue();
+    usePendingRecipients.mockReturnValue({
+      pendingRecipients: [],
+      setPendingRecipients: jest.fn(),
+      selectedOption: 'readWrite',
+      setSelectedOption: jest.fn()
+    });
+    getOrCreateFromArray.mockResolvedValue([]);
   });
 
   var setup = function setup() {
@@ -276,9 +241,7 @@ describe('FederatedFolderModal', function () {
         }
       }, _callee4);
     })));
-  });
-  describe('onShare callback', function () {
-    it('should update recipients when onShare is called', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee5() {
+    it('should pass selectedOption from usePendingRecipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee5() {
       var _setup5, getByTestId;
 
       return _regeneratorRuntime.wrap(function _callee5$(_context5) {
@@ -287,93 +250,83 @@ describe('FederatedFolderModal', function () {
             _setup5 = setup(), getByTestId = _setup5.getByTestId;
             _context5.next = 3;
             return waitFor(function () {
-              expect(getByTestId('recipients-count').textContent).toBe('0');
+              expect(getByTestId('selected-option').textContent).toBe('readWrite');
             });
 
           case 3:
-            fireEvent.click(getByTestId('btn-share'));
-            _context5.next = 6;
-            return waitFor(function () {
-              expect(getByTestId('recipients-count').textContent).toBe('2');
-            });
-
-          case 6:
           case "end":
             return _context5.stop();
         }
       }, _callee5);
     })));
-    it('should accumulate recipients when onShare is called multiple times', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee6() {
+  });
+  describe('onSend callback', function () {
+    var pendingRecipient = {
+      name: 'Alice',
+      email: 'alice@example.com'
+    };
+    it('should call onClose and skip share when there are no pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee6() {
       var _setup6, getByTestId;
 
       return _regeneratorRuntime.wrap(function _callee6$(_context6) {
         while (1) switch (_context6.prev = _context6.next) {
           case 0:
+            // pendingRecipients defaults to [] via beforeEach
             _setup6 = setup(), getByTestId = _setup6.getByTestId;
             _context6.next = 3;
             return waitFor(function () {
-              expect(getByTestId('recipients-count').textContent).toBe('0');
+              expect(getByTestId('btn-send')).toBeTruthy();
             });
 
           case 3:
-            fireEvent.click(getByTestId('btn-share'));
+            fireEvent.click(getByTestId('btn-send'));
             _context6.next = 6;
             return waitFor(function () {
-              expect(getByTestId('recipients-count').textContent).toBe('2');
+              expect(mockOnClose).toHaveBeenCalled();
+              expect(mockShare).not.toHaveBeenCalled();
             });
 
           case 6:
-            fireEvent.click(getByTestId('btn-share-other'));
-            _context6.next = 9;
-            return waitFor(function () {
-              expect(getByTestId('recipients-count').textContent).toBe('3');
-            });
-
-          case 9:
           case "end":
             return _context6.stop();
         }
       }, _callee6);
     })));
-  });
-  describe('onSend callback', function () {
     it('should call share with correct parameters', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee7() {
       var _setup7, getByTestId;
 
       return _regeneratorRuntime.wrap(function _callee7$(_context7) {
         while (1) switch (_context7.prev = _context7.next) {
           case 0:
+            usePendingRecipients.mockReturnValue({
+              pendingRecipients: [pendingRecipient],
+              setPendingRecipients: jest.fn(),
+              selectedOption: 'readWrite',
+              setSelectedOption: jest.fn()
+            });
+            getOrCreateFromArray.mockResolvedValue([pendingRecipient]);
             mockShare.mockResolvedValueOnce({});
             _setup7 = setup(), getByTestId = _setup7.getByTestId;
-            _context7.next = 4;
+            _context7.next = 6;
             return waitFor(function () {
               expect(getByTestId('btn-send')).toBeTruthy();
             });
 
-          case 4:
-            fireEvent.click(getByTestId('btn-share'));
+          case 6:
             fireEvent.click(getByTestId('btn-send'));
-            _context7.next = 8;
+            _context7.next = 9;
             return waitFor(function () {
               expect(mockShare).toHaveBeenCalledWith({
                 description: 'My Test Folder',
                 document: mockDocument,
-                recipients: [{
-                  _id: 'r1',
-                  id: 'r1',
-                  displayName: 'Alice'
-                }],
-                readOnlyRecipients: [{
-                  _id: 'r2',
-                  id: 'r2',
-                  displayName: 'Bob'
-                }],
+                recipients: [pendingRecipient],
+                readOnlyRecipients: [],
                 sharedDrive: true,
                 openSharing: false
               });
             });
 
-          case 8:
+          case 9:
           case "end":
             return _context7.stop();
         }
@@ -385,16 +338,23 @@ describe('FederatedFolderModal', function () {
       return _regeneratorRuntime.wrap(function _callee8$(_context8) {
         while (1) switch (_context8.prev = _context8.next) {
           case 0:
+            usePendingRecipients.mockReturnValue({
+              pendingRecipients: [pendingRecipient],
+              setPendingRecipients: jest.fn(),
+              selectedOption: 'readWrite',
+              setSelectedOption: jest.fn()
+            });
+            getOrCreateFromArray.mockResolvedValue([pendingRecipient]);
             mockShare.mockResolvedValueOnce({});
             _setup8 = setup(), getByTestId = _setup8.getByTestId;
-            _context8.next = 4;
+            _context8.next = 6;
             return waitFor(function () {
               expect(getByTestId('btn-send')).toBeTruthy();
             });
 
-          case 4:
+          case 6:
             fireEvent.click(getByTestId('btn-send'));
-            _context8.next = 7;
+            _context8.next = 9;
             return waitFor(function () {
               expect(mockShowAlert).toHaveBeenCalledWith({
                 message: 'Folder has been shared',
@@ -404,7 +364,7 @@ describe('FederatedFolderModal', function () {
               expect(mockOnClose).toHaveBeenCalled();
             });
 
-          case 7:
+          case 9:
           case "end":
             return _context8.stop();
         }
@@ -416,16 +376,23 @@ describe('FederatedFolderModal', function () {
       return _regeneratorRuntime.wrap(function _callee9$(_context9) {
         while (1) switch (_context9.prev = _context9.next) {
           case 0:
+            usePendingRecipients.mockReturnValue({
+              pendingRecipients: [pendingRecipient],
+              setPendingRecipients: jest.fn(),
+              selectedOption: 'readWrite',
+              setSelectedOption: jest.fn()
+            });
+            getOrCreateFromArray.mockResolvedValue([pendingRecipient]);
             mockShare.mockRejectedValueOnce(new Error('Share failed'));
             _setup9 = setup(), getByTestId = _setup9.getByTestId;
-            _context9.next = 4;
+            _context9.next = 6;
             return waitFor(function () {
               expect(getByTestId('btn-send')).toBeTruthy();
             });
 
-          case 4:
+          case 6:
             fireEvent.click(getByTestId('btn-send'));
-            _context9.next = 7;
+            _context9.next = 9;
             return waitFor(function () {
               expect(mockShowAlert).toHaveBeenCalledWith({
                 message: 'Error while sharing folder',
@@ -435,51 +402,45 @@ describe('FederatedFolderModal', function () {
               expect(mockOnClose).not.toHaveBeenCalled();
             });
 
-          case 7:
+          case 9:
           case "end":
             return _context9.stop();
         }
       }, _callee9);
     })));
   });
-  describe('onSetType callback', function () {
-    it('should move recipient from readOnlyRecipients to recipients when setting two-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10() {
+  describe('onRevoke callback', function () {
+    it('should call revoke from sharing context for existing members', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee10() {
       var _setup10, getByTestId;
 
       return _regeneratorRuntime.wrap(function _callee10$(_context10) {
         while (1) switch (_context10.prev = _context10.next) {
           case 0:
+            mockRevoke.mockResolvedValueOnce({});
             _setup10 = setup(), getByTestId = _setup10.getByTestId;
-            _context10.next = 3;
-            return waitFor(function () {
-              expect(getByTestId('btn-share')).toBeTruthy();
-            });
-
-          case 3:
-            fireEvent.click(getByTestId('btn-share')); // After btn-share: combined recipients = 2 (1 readWrite + 1 readOnly)
-
-            _context10.next = 6;
+            _context10.next = 4;
             return waitFor(function () {
-              expect(getByTestId('recipients-count').textContent).toBe('2');
-              expect(getByTestId('readOnly-recipients-count').textContent).toBe('1');
+              expect(getByTestId('dumb-modal')).toBeTruthy();
             });
 
-          case 6:
-            fireEvent.click(getByTestId('btn-set-type-two-way')); // After setting two-way: Bob moves from readOnlyRecipients to recipients
-
-            _context10.next = 9;
+          case 4:
+            fireEvent.click(getByTestId('btn-revoke'));
+            _context10.next = 7;
             return waitFor(function () {
-              expect(getByTestId('recipients-count').textContent).toBe('2');
-              expect(getByTestId('readOnly-recipients-count').textContent).toBe('0');
+              expect(mockRevoke).toHaveBeenCalledWith({
+                _id: 'folder-123'
+              }, 'abc', 1);
             });
 
-          case 9:
+          case 7:
           case "end":
             return _context10.stop();
         }
       }, _callee10);
     })));
-    it('should move recipient from recipients to readOnlyRecipients when setting one-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11() {
+  });
+  describe('onClose callback', function () {
+    it('should call onClose when close button is clicked', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee11() {
       var _setup11, getByTestId;
 
       return _regeneratorRuntime.wrap(function _callee11$(_context11) {
@@ -487,79 +448,6 @@ describe('FederatedFolderModal', function () {
           case 0:
             _setup11 = setup(), getByTestId = _setup11.getByTestId;
             _context11.next = 3;
-            return waitFor(function () {
-              expect(getByTestId('btn-share')).toBeTruthy();
-            });
-
-          case 3:
-            fireEvent.click(getByTestId('btn-share')); // After btn-share: combined recipients = 2 (1 readWrite + 1 readOnly)
-
-            _context11.next = 6;
-            return waitFor(function () {
-              expect(getByTestId('recipients-count').textContent).toBe('2');
-              expect(getByTestId('readOnly-recipients-count').textContent).toBe('1');
-            });
-
-          case 6:
-            fireEvent.click(getByTestId('btn-set-type-one-way')); // After setting one-way: Alice moves from recipients to readOnlyRecipients
-            // Combined count stays at 2, but readOnlyRecipients increases from 1 to 2
-
-            _context11.next = 9;
-            return waitFor(function () {
-              expect(getByTestId('recipients-count').textContent).toBe('2');
-              expect(getByTestId('readOnly-recipients-count').textContent).toBe('2');
-            });
-
-          case 9:
-          case "end":
-            return _context11.stop();
-        }
-      }, _callee11);
-    })));
-  });
-  describe('onRevoke callback', function () {
-    it('should remove recipient from both lists', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12() {
-      var _setup12, getByTestId;
-
-      return _regeneratorRuntime.wrap(function _callee12$(_context12) {
-        while (1) switch (_context12.prev = _context12.next) {
-          case 0:
-            _setup12 = setup(), getByTestId = _setup12.getByTestId;
-            _context12.next = 3;
-            return waitFor(function () {
-              expect(getByTestId('btn-share')).toBeTruthy();
-            });
-
-          case 3:
-            fireEvent.click(getByTestId('btn-share'));
-            _context12.next = 6;
-            return waitFor(function () {
-              expect(getByTestId('recipients-count').textContent).toBe('2');
-            });
-
-          case 6:
-            fireEvent.click(getByTestId('btn-revoke'));
-            _context12.next = 9;
-            return waitFor(function () {
-              expect(getByTestId('recipients-count').textContent).toBe('1');
-            });
-
-          case 9:
-          case "end":
-            return _context12.stop();
-        }
-      }, _callee12);
-    })));
-  });
-  describe('onClose callback', function () {
-    it('should call onClose when close button is clicked', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee13() {
-      var _setup13, getByTestId;
-
-      return _regeneratorRuntime.wrap(function _callee13$(_context13) {
-        while (1) switch (_context13.prev = _context13.next) {
-          case 0:
-            _setup13 = setup(), getByTestId = _setup13.getByTestId;
-            _context13.next = 3;
             return waitFor(function () {
               expect(getByTestId('btn-close')).toBeTruthy();
             });
@@ -570,9 +458,9 @@ describe('FederatedFolderModal', function () {
 
           case 5:
           case "end":
-            return _context13.stop();
+            return _context11.stop();
         }
-      }, _callee13);
+      }, _callee11);
     })));
   });
   describe('existing recipients', function () {
@@ -593,102 +481,24 @@ describe('FederatedFolderModal', function () {
       sharingId: 'abc',
       memberIndex: 1
     }];
-    it('should display existing recipients from getRecipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee14() {
-      var _setup14, getByTestId;
-
-      return _regeneratorRuntime.wrap(function _callee14$(_context14) {
-        while (1) switch (_context14.prev = _context14.next) {
-          case 0:
-            mockGetRecipients.mockReturnValue(existingRecipients);
-            _setup14 = setup(), getByTestId = _setup14.getByTestId;
-            _context14.next = 4;
-            return waitFor(function () {
-              expect(getByTestId('recipients-count').textContent).toBe('2');
-            });
-
-          case 4:
-          case "end":
-            return _context14.stop();
-        }
-      }, _callee14);
-    })));
-    it('should pass existing recipients as currentRecipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15() {
-      var _setup15, getByTestId;
+    it('should display existing recipients from getRecipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee12() {
+      var _setup12, getByTestId;
 
-      return _regeneratorRuntime.wrap(function _callee15$(_context15) {
-        while (1) switch (_context15.prev = _context15.next) {
+      return _regeneratorRuntime.wrap(function _callee12$(_context12) {
+        while (1) switch (_context12.prev = _context12.next) {
           case 0:
             mockGetRecipients.mockReturnValue(existingRecipients);
-            _setup15 = setup(), getByTestId = _setup15.getByTestId;
-            _context15.next = 4;
+            _setup12 = setup(), getByTestId = _setup12.getByTestId;
+            _context12.next = 4;
             return waitFor(function () {
               expect(getByTestId('recipients-count').textContent).toBe('2');
             });
 
           case 4:
           case "end":
-            return _context15.stop();
-        }
-      }, _callee15);
-    })));
-    it('should call revoke for existing member', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16() {
-      var _setup16, getByTestId;
-
-      return _regeneratorRuntime.wrap(function _callee16$(_context16) {
-        while (1) switch (_context16.prev = _context16.next) {
-          case 0:
-            mockGetRecipients.mockReturnValue(existingRecipients);
-            mockRevoke.mockResolvedValueOnce({});
-            _setup16 = setup(), getByTestId = _setup16.getByTestId;
-            _context16.next = 5;
-            return waitFor(function () {
-              expect(getByTestId('dumb-modal')).toBeTruthy();
-            });
-
-          case 5:
-            // Simulate revoke of an existing member (non-virtual-shared-drive index)
-            fireEvent.click(getByTestId('btn-revoke-existing'));
-            _context16.next = 8;
-            return waitFor(function () {
-              expect(mockRevoke).toHaveBeenCalledWith({
-                _id: 'folder-123'
-              }, 'abc', 1);
-            });
-
-          case 8:
-          case "end":
-            return _context16.stop();
-        }
-      }, _callee16);
-    })));
-  });
-  describe('createContact callback', function () {
-    it('should call client.create when creating a contact', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee17() {
-      var _setup17, getByTestId;
-
-      return _regeneratorRuntime.wrap(function _callee17$(_context17) {
-        while (1) switch (_context17.prev = _context17.next) {
-          case 0:
-            _setup17 = setup(), getByTestId = _setup17.getByTestId;
-            _context17.next = 3;
-            return waitFor(function () {
-              expect(getByTestId('btn-create-contact')).toBeTruthy();
-            });
-
-          case 3:
-            fireEvent.click(getByTestId('btn-create-contact'));
-            _context17.next = 6;
-            return waitFor(function () {
-              expect(client.create).toHaveBeenCalledWith('io.cozy.contacts', {
-                email: 'test@example.com'
-              });
-            });
-
-          case 6:
-          case "end":
-            return _context17.stop();
+            return _context12.stop();
         }
-      }, _callee17);
+      }, _callee12);
     })));
   });
 });

package/dist/components/Recipient/GroupRecipientPermissions.js

@@ -2,17 +2,18 @@ import _asyncToGenerator from "@babel/runtime/helpers/asyncToGenerator";
 import _slicedToArray from "@babel/runtime/helpers/slicedToArray";
 import _regeneratorRuntime from "@babel/runtime/regenerator";
 import React, { useState, useRef } from 'react';
-import { useClient } from 'cozy-client';
 import ActionsMenu from 'cozy-ui/transpiled/react/ActionsMenu';
-import { makeActions, divider } from 'cozy-ui/transpiled/react/ActionsMenu/Actions';
+import { makeActions } from 'cozy-ui/transpiled/react/ActionsMenu/Actions';
 import ActionsMenuMobileHeader from 'cozy-ui/transpiled/react/ActionsMenu/ActionsMenuMobileHeader';
 import DropdownButton from 'cozy-ui/transpiled/react/DropdownButton';
+import Icon from 'cozy-ui/transpiled/react/Icon';
+import IconButton from 'cozy-ui/transpiled/react/IconButton';
+import CrossCircleOutlineIcon from 'cozy-ui/transpiled/react/Icons/CrossCircleOutline';
 import ListItemIcon from 'cozy-ui/transpiled/react/ListItemIcon';
 import ListItemText from 'cozy-ui/transpiled/react/ListItemText';
 import Spinner from 'cozy-ui/transpiled/react/Spinner';
 import { useI18n } from 'twake-i18n';
 import { permission } from './actions/permission';
-import { revokeGroup as revokeGroupAction } from './actions/revokeGroup';
 import { useSharingContext } from '../../hooks/useSharingContext';
 import { GroupAvatar } from '../Avatar/GroupAvatar';
 
@@ -31,7 +32,6 @@ var GroupRecipientPermissions = function GroupRecipientPermissions(_ref) {
       t = _useI18n.t;
 
   var buttonRef = useRef();
-  var client = useClient();
 
   var _useSharingContext = useSharingContext(),
       revokeGroup = _useSharingContext.revokeGroup,
@@ -96,12 +96,9 @@ var GroupRecipientPermissions = function GroupRecipientPermissions(_ref) {
   }();
 
   var type = isReadOnly ? 'one-way' : 'two-way';
-  var actions = makeActions([permission, divider, revokeGroupAction], {
-    client: client,
+  var actions = makeActions([permission], {
     t: t,
-    type: type,
-    isOwner: isOwner,
-    handleRevocation: handleRevocation
+    type: type
   });
   return /*#__PURE__*/React.createElement("div", {
     className: className
@@ -111,7 +108,7 @@ var GroupRecipientPermissions = function GroupRecipientPermissions(_ref) {
     "aria-haspopup": "true",
     onClick: toggleMenu,
     textVariant: "body2"
-  }, t("Share.type.".concat(type)).toLowerCase()), /*#__PURE__*/React.createElement(ActionsMenu, {
+  }, t("Share.type.".concat(type))), /*#__PURE__*/React.createElement(ActionsMenu, {
     ref: buttonRef,
     open: isMenuDisplayed,
     actions: actions,
@@ -124,7 +121,14 @@ var GroupRecipientPermissions = function GroupRecipientPermissions(_ref) {
     primaryTypographyProps: {
       variant: 'h6'
     }
-  })))));
+  }))), /*#__PURE__*/React.createElement(IconButton, {
+    onClick: handleRevocation,
+    size: "small",
+    className: "u-ml-half",
+    "aria-label": t('Share.members.revoke')
+  }, /*#__PURE__*/React.createElement(Icon, {
+    icon: CrossCircleOutlineIcon
+  }))));
 };
 
 export { GroupRecipientPermissions };