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

Control user button issue #3785

Closed
S-Haime opened this issue Nov 14, 2023 · 5 comments · Fixed by #3891
Closed

Control user button issue #3785

S-Haime opened this issue Nov 14, 2023 · 5 comments · Fixed by #3891
Assignees
Labels
bug Something's not working

Comments

@S-Haime
Copy link
Member

S-Haime commented Nov 14, 2023

Reported by Longleaf (ticket #7466)

"When I go into the Manager role and to the "Enrolled Users" area of any journal, the "Control User" button in the first row of the list of users appears to be functioning as the "Add Users to Role" button. When I clicked "Control User" in the row for a user (see screenshot below), I got the error message "No users selected," which is what happens if I click "Add Users to Role" without selecting a box.

The "Control User" buttons for all other rows work as expected. It's just happening in the first row. Could you please look into this apparent bug?
image

@S-Haime S-Haime added the bug Something's not working label Nov 14, 2023
@S-Haime
Copy link
Member Author

S-Haime commented Dec 12, 2023

Hi @StephDriver - the user was not trying to control themselves and they were not the first users on the list (I checked).

@StephDriver
Copy link
Contributor

Investigation:
problem is related to lowest user.pk rather than position in table. When tested with two users (pk 1, 2) the 1 showed this bug whether in top line of user table or bottom. 2 didn't show the bug in either position. pk appears to be correct on each line of table.

next step -
test with multiple users were lowest user.pk is not 1 as to whether this is specific to pk=1 or whether it is about the lowest pk.

@S-Haime
Copy link
Member Author

S-Haime commented Apr 26, 2024

So this still seems to be an issue... See FD

@S-Haime S-Haime reopened this Apr 26, 2024
@ajrbyers
Copy link
Member

I don’t think it’s been deployed to UNC yet.

@S-Haime
Copy link
Member Author

S-Haime commented Apr 26, 2024

I shall inform Stacy!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something's not working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants