-
Notifications
You must be signed in to change notification settings - Fork 38
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
Update the UI to switch user aka become user #4294
base: main
Are you sure you want to change the base?
Conversation
This looks and works really nicely! Will be a pleasure to use when we get it deployed. I have one issue. The function With 4000 users it takes around three (3) seconds. I think OTF has even more users. This should not be used in production but even test data can have a lot of users. Possible to speed it up a bit, showing the role in the list is useful so good if we can keep that. Maybe caching the list would work? |
Will look into it. |
This is super slick! I really like the banner as opposed to the pop up window that you need to dismiss every time you reload the page. Nice improvement! |
@frjo Updated the query to fetch the groups. It should always come down to 2 queries now. That said, if we face issue in the test site, we can switch to fetching the options as you type, via a search api. |
The new implementation of |
33c3b69
to
8487108
Compare
- Rename to "switch user" - Can switch from the user menu - Switch to choices.js from Select2 - Update the hijacked view, so an alert at the top
8487108
to
9bf14fb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now the load time is more or less instant and works equally well in other respects.
Fixes #4131