Skip to content
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

Closed
eaquigley opened this issue Jul 9, 2014 · 5 comments
Assignees
Labels
Type: Feature a feature request

Comments

@eaquigley
Copy link
Contributor


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.

@eaquigley eaquigley added this to the Dataverse 4.0: In Review milestone Jul 9, 2014
@eaquigley eaquigley mentioned this issue Jul 9, 2014
@scolapasta scolapasta modified the milestones: Beta 4 - Dataverse 4.0, In Review - Dataverse 4.0 Jul 15, 2014
@scolapasta scolapasta modified the milestones: Beta 4 - Dataverse 4.0, Beta 7 - Dataverse 4.0 Aug 19, 2014
@scolapasta
Copy link
Contributor

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.

@kcondon
Copy link
Contributor

kcondon commented Nov 16, 2014

@scolapasta Can you give Elda and me another overview based on the current status? Thx

@kcondon kcondon assigned scolapasta and unassigned kcondon Nov 25, 2014
@kcondon
Copy link
Contributor

kcondon commented Nov 25, 2014

Passing back until "all clear"

@scolapasta
Copy link
Contributor

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.

@scolapasta scolapasta removed their assignment Nov 26, 2014
@kcondon kcondon self-assigned this Dec 1, 2014
@kcondon
Copy link
Contributor

kcondon commented Dec 2, 2014

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.

@kcondon kcondon closed this as completed Dec 2, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature a feature request
Projects
None yet
Development

No branches or pull requests

3 participants