Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #12362 (closed)

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

BUG: Change permissions - group owner

Reported by: pwalczysko Owned by: mtbcarroll
Priority: major Milestone: 5.0.3
Component: Insight Version: 5.0.2
Keywords: n.a. Cc: ux@…,
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: n.a.

Description (last modified by mtbcarroll)

Noticed by Tristan during Paris meeting.

In INsight, there is no possibility to change group permission level for group owner, menu is greyed out.

In web, this functionality is possible, although downgrade to Private is always greyed out. See #12363.

Change History (7)

comment:1 Changed 5 years ago by mtbcarroll

  • Owner changed from jburel to mtbcarroll

Group owners should be able to change group permissions (except down to private), right?

comment:2 Changed 5 years ago by wmoore

They should be allowed to *try* and change group permissions, even down to private.
But this may fail (E.g. if links exist that would be incorrectly severed) so in web we say "You *may* be able to change group permission to Private" and then handle the exception nicely. See https://github.com/openmicroscopy/openmicroscopy/pull/2485
Just need to allow the same thing for group owner in web, and in Insight.

comment:3 Changed 5 years ago by mtbcarroll

  • Component changed from Insight to Web
  • Owner changed from mtbcarroll to web-team@…

Fixed in Insight by https://github.com/openmicroscopy/openmicroscopy/pull/2655. Just minor web to-do now?

comment:4 Changed 5 years ago by pwalczysko

@mtbcarroll : Web has its own ticket for this see http://trac.openmicroscopy.org.uk/ome/ticket/12363

comment:5 Changed 5 years ago by mtbcarroll

  • Component changed from Web to Insight
  • Description modified (diff)
  • Owner changed from web-team@… to mtbcarroll
  • Status changed from new to accepted

Ah, perfect, thank you Petr.

comment:6 Changed 5 years ago by mtbcarroll

  • Resolution set to fixed
  • Status changed from accepted to closed

comment:7 Changed 5 years ago by Josh Moore <josh@…>

  • Remaining Time set to 0

(In [6a2150ee9784833b09cb5b1ec1f4b66904d08118/ome.git] on branch develop) Merge pull request #2752 from mtbc/trac-12362-group-owner-change-permissions

fix #12362, #12396: allow group owner to change permission level (rebased from dev_5_0)

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.78256 sec.)

We're Hiring!