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

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

Closed
nicolasbrossard opened this issue Nov 26, 2019 · 2 comments · Fixed by #8967
Closed
Assignees
Labels
Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) Needs Maintainer PR owner is no longer involved in it. PR is up for adoption by a new developer

Comments

@nicolasbrossard
Copy link
Contributor

I created a user on the test VM that is an examiner at all the sites (MTL, OTT, ROM and DCC). I then edited the user profile so that he was only an examiner at DCC, with 'Radiologist' set to 'Yes'. When I access the examiner module, there is no indication that the user has been inactivated at sites MTL, OTT and ROM. In fact the table shows that the user os a Radiologist at every site. I also checked the backend examiner_psc_rel table and it did not contain any inactive examiners.

@nicolasbrossard nicolasbrossard added 22.0.0 TESTING Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) labels Nov 26, 2019
@kchatpar
Copy link

@ridz1208 said to close the issue and there's no current solution

@ridz1208 ridz1208 reopened this Dec 16, 2019
@ridz1208
Copy link
Collaborator

@nicolasbrossard I did say that !!

however while re-reading the issue, the last part might be an actual bug. the backend table should show the inactive flag.

@kchatpar please investigate that part

@johnsaigle johnsaigle added the Needs Maintainer PR owner is no longer involved in it. PR is up for adoption by a new developer label Mar 10, 2020
@CamilleBeau CamilleBeau self-assigned this Nov 14, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) Needs Maintainer PR owner is no longer involved in it. PR is up for adoption by a new developer
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants