volto icon indicating copy to clipboard operation
volto copied to clipboard

Tests groupmembership panel

Open ksuess opened this issue 1 year ago • 7 comments

Needs plone.restapi>=9.3.0 (# 1712 Give Site Administrator permission to manage users)

ON HOLD until DOCKER_IMAGE_ACCEPTANCE includes plone.restapi>=9.3.0 https://github.com/plone/volto/blob/97f830a4c7919f93904985b54745c586a31c9a5f/variables.mk#L3

ksuess avatar Feb 26 '24 10:02 ksuess

Deploy Preview for volto ready!

Name Link
Latest commit c88aea07e499175663aa99e922e214e568600c1e
Latest deploy log https://app.netlify.com/sites/volto/deploys/65dc6b893f139c0008ac907a
Deploy Preview https://deploy-preview-5801--volto.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Feb 26 '24 10:02 netlify[bot]

Deploy Preview for plone-components ready!

Name Link
Latest commit c88aea07e499175663aa99e922e214e568600c1e
Latest deploy log https://app.netlify.com/sites/plone-components/deploys/65dc6b897dd45d0008cb4333
Deploy Preview https://deploy-preview-5801--plone-components.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Feb 26 '24 10:02 netlify[bot]

Need some help with cy.autologin for non-manager users.

describe('User Group Membership Control Panel test for non-manager', () => {
  beforeEach(() => {
    init();
    cy.createUser({
      username: 'siteadmin',
      fullname: 'Sven Siteadministrator',
      roles: ['Site Administrator'],
    });
    cy.autologin('siteadmin', 'password');
  });

Now with role Site Administrator the user should be granted the right to use the group membership panel, which is at least fetching Plone restapi @users and @groups.

ksuess avatar Feb 26 '24 12:02 ksuess

@ksuess there is still a problem with Site Adminstrator add users to groups. To see:

https://github.com/plone/plone.restapi/pull/1751

There has not yet been a release of plone.restapi with this fix.

wesleybl avatar Feb 26 '24 14:02 wesleybl

@tisto Can we get a release of plone.restapi, please, to proceed with #5244.

ksuess avatar Feb 26 '24 15:02 ksuess

@ksuess I released plone.restapi 9.5.0.

tisto avatar Feb 27 '24 06:02 tisto

@ksuess plone.restapi 9.5.0 is included in Plone 6.0.10.1 which has Docker images available today. :)

davisagli avatar Feb 29 '24 01:02 davisagli