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

Permission improvements of the submission archive feature #3389

Closed
4 tasks
frjo opened this issue May 3, 2023 · 3 comments · Fixed by #3646
Closed
4 tasks

Permission improvements of the submission archive feature #3389

frjo opened this issue May 3, 2023 · 3 comments · Fixed by #3646
Assignees
Labels
Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature).
Milestone

Comments

@frjo
Copy link
Member

frjo commented May 3, 2023

Here are some issue with the current implementation:

  • All staff can archive an submission, if they then do not have permission to view archived submission they get a 403. If they should be able to do this a redirect would be nicer.
  • Should all staff be able to archive a submission?
  • Be default no one can see archived submission, SUBMISSIONS_ARCHIVED_ACCESS_STAFF and SUBMISSIONS_ARCHIVED_ACCESS_STAFF_ADMIN are both false be default. Either super admins should have access or SUBMISSIONS_ARCHIVED_ACCESS_STAFF_ADMIN should default to true.
  • Should we show on archived submission who can access them, maybe in the red border add info if staff or staff admins only can see this?
@frjo frjo added the Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature). label May 3, 2023
@fourthletter fourthletter added this to the Backlog milestone Sep 22, 2023
@wes-otf
Copy link
Contributor

wes-otf commented Nov 8, 2023

Might make sense for me to wrap these into #3646, four birds one stone. After talking with @Techslammer it seems like it'd be best for OTF's workflow if the only role that could make & access archives would be Staff Admin. @frjo what are your thoughts on that?

@frjo
Copy link
Member Author

frjo commented Nov 9, 2023

I think it makes sense for the default to be that Staff Admins can archive and see archived submissions. Then we have settings that can be turned on that allow staff to do the archiving and see the archives.

So default settings like these, two new added and two renamed for clarity:

SUBMISSIONS_ARCHIVED_VIEW_ACCESS_STAFF = False
SUBMISSIONS_ARCHIVED_VIEW_ACCESS_STAFF_ADMIN = True 
SUBMISSIONS_ARCHIVE_ACCESS_STAFF = False
SUBMISSIONS_ARCHIVE_ACCESS_STAFF_ADMIN = True

By settings all to false an organisation can then turn off the feature if they want.

Feel free to improve the settings names!

@wes-otf
Copy link
Contributor

wes-otf commented Nov 13, 2023

All of this sounds great! What did you have in mind for the redirects? In my current implementation I just have it redirect the user to the submission list page. Should we have a dedicated view for any permission denied?

@frjo frjo closed this as completed in #3646 Dec 1, 2023
frjo pushed a commit that referenced this issue Dec 1, 2023
Closes #3388 & Closes #3389.

This PR adds an indication as to what user roles can see an archived
submission based off of [existing
settings](#3388 (comment)).
wes-otf added a commit that referenced this issue May 7, 2024
Closes #3388 & Closes #3389.

This PR adds an indication as to what user roles can see an archived
submission based off of [existing
settings](#3388 (comment)).
wes-otf added a commit that referenced this issue May 8, 2024
Closes #3388 & Closes #3389.

This PR adds an indication as to what user roles can see an archived
submission based off of [existing
settings](#3388 (comment)).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature).
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants