-
Notifications
You must be signed in to change notification settings - Fork 487
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Permissions: Implement permissions for all existing features, incl, create/edit/delete dv, datasets. #650
Comments
Current functionality should all be using permissions and commands. If you find anything not working, please open as a new (specific) ticket. Now that there is a UI for roles and permissions will be easier to check. |
@scolapasta Can you give Elda and me another overview based on the current status? Thx |
Passing back until "all clear" |
we should be able to test anything if the current required permissions. There are still some things that need to be cleaned up / tweaked related to assiging roles or what roles has what permission, but the core logic: If you require perm A, and you have that permission via some role you have, then you should be able to perform the action. Otherwise, you should not. |
Basic permissions have been implemented with the exception of inheritance. There is some design work still happening but the primary functionality is there and issues will be opened as separate tickets. |
Author Name: Kevin Condon (@kcondon)
Original Redmine Issue: 4100, https://redmine.hmdc.harvard.edu/issues/4100
Original Date: 2014-06-16
Original Assignee: Gustavo Durand
We are at the point where we should be using permissions for existing functionality to make development more complete and ready and testing more comprehensive. Also, users will see how it will really work.
I think this is better done sooner than later so we don't have unexpected issues late in the schedule.
The text was updated successfully, but these errors were encountered: