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

Make it clear who sees an archived submission #3388

Closed
fourthletter opened this issue May 2, 2023 · 4 comments · Fixed by #3646
Closed

Make it clear who sees an archived submission #3388

fourthletter opened this issue May 2, 2023 · 4 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

@fourthletter
Copy link
Contributor

Description

As a user I would need to know who gets to see that this submission has been archived. Other submissions have 'team' or 'applicant' with an eye icon.

image

@fourthletter fourthletter added the Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature). label May 2, 2023
@frjo
Copy link
Member

frjo commented May 3, 2023

The icons indicates who can see the messages, it does not related to the content of the message. Mostly relevant for comments.

We use the same templates for activity and comments so some activities get the icons to. I do not think we really need them there, see no purpose.

Showing who can see archived submissions should be solved somewhere else.

Currently there are three option for who can see archived submissions. By default, no one can see them. Each organisation can make setting that make them available to "Staff admin" and/or "Staff" user roles.

Maybe we can add some text to the red banner?

@frjo
Copy link
Member

frjo commented May 3, 2023

I have created a bigger issue with improvements to the archive feature, if it covers your issue you can close this one.

#3389

@fourthletter fourthletter added this to the Backlog milestone Sep 22, 2023
@wes-otf
Copy link
Contributor

wes-otf commented Nov 3, 2023

@frjo what permissions do I need to set to recreate this in my local dev environment? I went through the process of:

  1. Set my superuser group to Staff & Staff Admin
  2. Setting up a new round & fund
  3. Created another account as an applicant and applied for said fund
  4. Back on my superuser account, archived the submission

I was then redirected to the dashboard, and when attempting to select the fund I get a "403 Forbidden" screen. Did I miss a step?

@frjo
Copy link
Member

frjo commented Nov 5, 2023

By default only superadmins can see archived submissions. See these settings to change that.

# Provide permissions for Archived submissions
SUBMISSIONS_ARCHIVED_ACCESS_STAFF = env.bool("SUBMISSIONS_ARCHIVED_ACCESS_STAFF", False)
SUBMISSIONS_ARCHIVED_ACCESS_STAFF_ADMIN = env.bool(
    "SUBMISSIONS_ARCHIVED_ACCESS_STAFF_ADMIN", False
)

Other user roles should never see archived submissions.

@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