#3598 Improve project permissions admin

v1.0.0
closed
sf-4 (350)
Admin
2015-08-20
2012-01-17
No

During testing, users were very confused by the usergroup and permissions screens. They had trouble figuring out how to add a user to their project and what the various permissions meant. One suggestion was to combine the screens and explain how the two areas interacted. Whatever we decide to do here, we should add much more explanatory text about how the permissions system works. If we do keep the pages separate, there should be a link on each explaining that some related changes need to be made on the other page.

Discussion

1 2 > >> (Page 1 of 2)
  • Dave Brondsema

    Dave Brondsema - 2012-01-20
    • size: --> 4
     
  • Dave Brondsema

    Dave Brondsema - 2012-01-20
    • combine groups & permissions page into one, for the project level
    • remove the cards
    • replace them with a table and use rows to list the group or permission, and its memberships
    • tool permissions should
     
  • Dave Brondsema

    Dave Brondsema - 2012-03-23

    We also need to make subproject permission page continue to work. Subprojects just have a permission page, no groups page (similar to tools)

     
  • Jenny Steele - 2012-04-06
    • status: open --> in-progress
    • assigned_to: Jenny Steele
     
  • Jenny Steele - 2012-04-13
    • status: in-progress --> code-review
    • qa: ⸘Dave Brondsema‽
     
  • Jenny Steele - 2012-04-13

    On allura and sftheme js/3598. To test, go to the User Permissions page in project admin. You should be able to create/delete groups, add/remove users in groups, and add/remove permissions in groups. Check that the explanatory text makes sense. Old-style permission management for tools and subprojects should still work.

     
  • Jenny Steele - 2012-04-16

    Once this is ready, I could do a write-up on it for [#3801]. A full explanation of the Allura permissions system might be interesting as a possible separate post too.

     
  • Dave Brondsema

    Dave Brondsema - 2012-04-17
    • status: code-review --> open
     
  • Dave Brondsema

    Dave Brondsema - 2012-04-17

    On a pre-existing or new project, I get this error on the project's /admin/groups/ page, as well as the neighborhood's /p/admin/groups/ and some of the TestProjectAdmin tests

    File '/home/dbrondsema/dbrondsema-7024/forge/Allura/allura/ext/admin/admin_main.py', line 649 in index
      permissions_by_role = self._map_group_permissions()
    File '/home/dbrondsema/dbrondsema-7024/forge/Allura/allura/ext/admin/admin_main.py', line 612 in _map_group_permissions
      for r in role.child_roles():
    AttributeError: 'ProjectRole' object has no attribute 'child_roles'
    
     
  • Jenny Steele - 2012-04-18
    • status: open --> code-review
     
  • Jenny Steele - 2012-04-18

    Looks like some code didn't get checked in. Try it now.

     
  • Jenny Steele - 2012-04-19

    Team feedback:

    • Add red/green color to permission icons
    • Make minus icon for user remove always visible
     
  • Jenny Steele - 2012-04-19

    Updated with the changes above.

     
    • qa: Dave␣Brondsema --> Tim Van Steenburgh
     
    • status: code-review --> in-progress
     
  • Using the default 'test' project on a sandbox. After adding 3 users to the Developer group, I clicked the Delete icon next to the first user in the list, but it was the second user that was actually deleted.

     
  • Jenny Steele - 2012-04-24
    • status: in-progress --> code-review
     
  • Jenny Steele - 2012-04-24

    Fixed the user delete problem.

     
    • status: code-review --> in-progress
     
  • I'm still having the same problem. I deleted my local js/3598, checked it out from origin again, and cleared my browser cache.

    I added a new group. Added a user to that group. Clicked on that user to delete it, but instead, the new user text box appeared. So I added another user. Then clicked on the first user to delete it, but instead the second user was deleted.

    I also noticed that if I click on an inherited permission, I get a spinner that never goes away (unless I refresh the page).

     
  • Jenny Steele - 2012-04-26

    Fixes added for these problems.

     
  • Jenny Steele - 2012-04-26
    • status: in-progress --> code-review
     
    • When I enable a perm on the Member group, the Develop group inherits it correctly, but the Admin group doesn't.
    • If I click to try to remove the admin perm from the Admin group, I get a flash warning that it's not allowed (which is good), but I also get an infinite spinner on the admin perm.
     
    • status: code-review --> in-progress
     
  • Jenny Steele - 2012-04-26
    • status: in-progress --> code-review
     
1 2 > >> (Page 1 of 2)

Log in to post a comment.