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.

Sign up for GitHub

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

Jump to bottom

Remove exporter from the user roles #933

Closed
TanyaStere42 opened this issue Nov 4, 2023 · 2 comments · Fixed by #941
Closed

Remove exporter from the user roles #933

TanyaStere42 opened this issue Nov 4, 2023 · 2 comments · Fixed by #941
Assignees
@naknomum
Labels
enhancement New feature or request
Milestone
Codex Release 2.2

Comments

@TanyaStere42
Copy link
Member

Context

If exporter role is now a collaboration, it doesn't need to be a user role

User Story

  • As a user manager, I can only assign these four roles:
    • Contributor
    • Researcher
    • User manager
    • Admin

Technical note

Migration concern?: Anyone with researcher and without exporter becomes a researcher. This should work as expected.
Anyone with exporter will now have restricted access to data they were able to export previously. This is expected and should encourage increased collaborations

Dependent Stories

Very intertwined with Fold export permissions into collaboration hierarchy
@TanyaStere42 TanyaStere42 added this to the Codex Release 2.2 milestone Nov 4, 2023
@TanyaStere42 TanyaStere42 added the enhancement New feature or request label Nov 4, 2023
@TanyaStere42
Copy link
Member Author

Verify this is true in the FE listing of roles automagically

@TanyaStere42 TanyaStere42 mentioned this issue Nov 4, 2023
Open
@TanyaStere42 TanyaStere42 mentioned this issue Nov 27, 2023
Closed
naknomum added a commit that referenced this issue Dec 13, 2023
@naknomum
drop export-specific permissions stuff (issue #933)
c1cb97d
naknomum added a commit that referenced this issue Dec 14, 2023
@naknomum
drop export-specific permissions stuff (issue #933)
5acecd4
@naknomum naknomum mentioned this issue Dec 14, 2023
Merged
@naknomum naknomum linked a pull request Dec 14, 2023 that will close this issue
alter export to respect per-object permissions #941
Merged
@naknomum naknomum self-assigned this Dec 14, 2023
naknomum added a commit that referenced this issue Dec 15, 2023
@naknomum
drop export-specific permissions stuff (issue #933)
ea9a429
@TanyaStere42 TanyaStere42 mentioned this issue Dec 19, 2023
Closed
@TanyaStere42
Copy link
Member Author

QA:

  • user role creation shows the restricted list
  • user role edit shows the restricted list
  • users have been correctly migrated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@naknomum naknomum

Labels
enhancement New feature or request
Projects
No open projects
Codex 2.2.0
Status: Done
Milestone
Codex Release 2.2
Development

Successfully merging a pull request may close this issue.

alter export to respect per-object permissions WildMeOrg/houston
2 participants
@naknomum @TanyaStere42