Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
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 10 years ago

Closed 10 years ago

Last modified 10 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 10 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 10 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 10 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 10 years ago by pwalczysko

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

comment:5 Changed 10 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 10 years ago by mtbcarroll

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

comment:7 Changed 10 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.64231 sec.)

We're Hiring!