Skip to content

Conversation

CamilleBeau
Copy link
Collaborator

Brief summary of changes

This PR modifies the user accounts module to have a multi-select field for examiner sites instead of checkboxes. This way the full site names are listed and a user can see more clearly what sites they are selecting for an examiner. All other behaviour of examiner selection should remain the same.

image
  • Have you updated related documentation?

Testing instructions (if applicable)

  1. Go to a user who is not an examiner yet.
  2. Check that the validation for examiner fields works: the examiner site field should be required if radiologist and/or pending is selected, and likewise the radiologist / pending field should be required if an examiner field is selected
  3. Save the user with 1 or 2 examiner fields selected, as well as the radiologist / pending fields selected
  4. Navigate away from the page, refresh, and go back to edit the user. Make sure the same sites are still displayed as selected
  5. Remove one site and save the user. Navigate away, refresh, then come back. Make sure the site was properly removed
  6. Repeat the above step to remove all sites (make sure to also unset the radiologist / pending values). Make sure the sites were properly removed.
  7. Try editing a user from a user who has does not have access to all sites. Make sure that only the current user's sites are displayed in the examiner sites multiselect, and make sure that no extra sites are added / removed for the examiner when edited.

Link(s) to related issue(s)

@CamilleBeau CamilleBeau added the Module: user_accounts PR or issue related to the user_accounts module label Oct 20, 2025
@github-actions github-actions bot added the Language: PHP PR or issue that update PHP code label Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language: PHP PR or issue that update PHP code Module: user_accounts PR or issue related to the user_accounts module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[User account] - Examiner at selection

1 participant