U4-10083 - Wire up granular permissions in the user group editor in c#

Created by Shannon Deminick 30 Jun 2017, 07:22:23 Updated by Claus Jensen 05 Jul 2017, 12:23:28

Tags: Unscheduled

Relates to: U4-10109

Subtask of: U4-8632

Your report will have a greater chance of being addressed if you can give us clear steps to reproduce the issue, please answer the following questions in as much detail as possible: What did you do?

What did you expect to happen?

What actually happened?

Comments

Shannon Deminick 04 Jul 2017, 05:43:38

For testing:

  • Go to edit a user group
  • at the bottom you can add granular permissions to content items
  • Press add, select a content item, select some permissions (the default permissions of the group will be displayed)
  • Select ok for everything and save the group - it should be saved and show the selected permissions
  • Refresh to make sure it's really saved
  • Do the same thing for a different content item
  • Do the same thing for an already chosen content item - this will essentially perform an edit
  • Go to one of the content nodes in the content tree that you assigned permissions to and select the 'Permissions' context menu, you should see the permissions assigned there too


Claus Jensen 05 Jul 2017, 10:24:07

All works - marking as fixed.

But I'm not sure if we should consider changing the 'Add' button to say 'Add or edit' or something similar?

It is really cool that you can edit the existing permissions of a node by simply "adding" it again. The problem is, I would never have known this unless it said to test it here in this issue. I only see 'Add' and 'Remove' as action options in the UI, so my first move would be to simply click remove and then re-add the item - never knowing that I could actually edit it instead.

@madsrasmussen @Shandem what do you guys think?


Shannon Deminick 05 Jul 2017, 12:18:13

Yup, i think on my commit I had the same comment. I thought I had created a task for this but maybe not, @claus could you create one please?


Claus Jensen 05 Jul 2017, 12:23:28

Yep :) done http://issues.umbraco.org/issue/U4-10109


Priority: Normal

Type: Task

State: Fixed

Assignee:

Difficulty: Normal

Category:

Backwards Compatible: True

Fix Submitted:

Affected versions:

Due in version:

Sprint: Sprint 62

Story Points: 3

Cycle: