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

[examiner] Filter out sites user does not have access to #9728

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

CamilleBeau
Copy link
Contributor

Brief summary of changes

This PR restricts the examiner sites shown in the examiner module to only the sites that the user has access to in the case that the user does not have the "All sites" permission for the examiner module.

  • Have you updated related documentation?

Testing instructions (if applicable)

  1. Create a user with the "Examiner: Add and Certify Examiners - Own Sites" permission. Give this user access to some but not all sites
  2. Access the examiner module with that user
  3. Make sure that the module loads and that the "Site" column in the filterable datatable only includes the sites that the user has access to
  4. Give the user the "Examiner: Add and Certify Examiners - All Sites" permission
  5. Make sure that the "Site" column now shows all sites, not only the sites that the user has permission to.

Link(s) to related issue(s)

@CamilleBeau CamilleBeau added the 27.0.0 - Bugs Bugs Found in LORIS 27 testing label Mar 26, 2025
@CamilleBeau CamilleBeau added this to the 27.0.0 milestone Mar 26, 2025
@CamilleBeau CamilleBeau added the Module: examiner PR or issue related to examiner module label Mar 26, 2025
@driusan
Copy link
Collaborator

driusan commented Mar 27, 2025

@GeorgeMurad do you have time to test/review?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
27.0.0 - Bugs Bugs Found in LORIS 27 testing Module: examiner PR or issue related to examiner module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[examiner] Blank rendered for site user not part of
3 participants