-
Notifications
You must be signed in to change notification settings - Fork 174
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
[data_release] Introduce Project Separation #9385
base: main
Are you sure you want to change the base?
[data_release] Introduce Project Separation #9385
Conversation
@shonibare sorry about that! resolved, but make sure you compile because the manage permissions form should look like this, not the screenshot |
@skarya22, you are right. I retested it again, and it produced the expected result. A user can not manage permissions or see files not affiliated with its site. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Brief summary of changes
Testing instructions (if applicable)
Data Release: Grant Other Users Access to Releases permission
Related to CCNR PR 7445