Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[examiner] Only show active examiner sites #8967

Merged
merged 3 commits into from
Dec 6, 2023

Conversation

CamilleBeau
Copy link
Contributor

@CamilleBeau CamilleBeau commented Nov 21, 2023

Brief summary of changes

In this PR, only active examiner sites are shown for an examiner in the examiner module. Previously, the module was showing any site that the examiner had once been an examiner at, whether it was active or not.

  • Have you updated related documentation?

Testing instructions (if applicable)

  1. Add a user as an examiner setting several examiner sites
  2. Remove some sites from the examiner
  3. Load the examiner module and search for the user
  4. Make sure that only sites that are still active are shown in the examiner module

Link(s) to related issue(s)

@driusan driusan merged commit ed5df9c into aces:main Dec 6, 2023
19 checks passed
@KLaFleur KLaFleur added the Passed Manual Tests PR has undergone proper testing by at least one peer label Dec 6, 2023
jeffersoncasimir added a commit to jeffersoncasimir/Loris that referenced this pull request Jan 23, 2024
@ridz1208 ridz1208 added this to the 26.0.0 milestone Apr 9, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Passed Manual Tests PR has undergone proper testing by at least one peer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[User account] Removing sites at which a user is an examiner does not seem to work
4 participants