Fix superuser dashboard recently added users #3473
Assignees
Labels
Help Wanted
Groomed + open to all!
Comments
|
Hey, I would like to work on it! |
|
Go for it! |
1 task
|
This issue has been inactive for 245 hours (10.21 days) and will be automatically unassigned after 115 more hours (4.79 days). |
|
Hi, I didn't have time to work on this issue lately, but I started to work on it today! If the GitHub actions unassign me, could you please give it to me again? Thank you for being patient. |
|
This issue has been inactive for 251 hours (10.46 days) and will be automatically unassigned after 109 more hours (4.54 days). |
dorner
pushed a commit
that referenced
this issue
Apr 19, 2023
* add email to the text and change link to edit_user_admin rather than admin_user * add a limit of 20 to recent users query * refactor recent_users query * add specs for recent users on admin dashboard * add helper to display correct text for a user that does not have a name * merge three it test cases into a single one to avoid overdoing user creatioon * removing unnecessary `.presence` method call
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Summary
Make the superuser dashboard "recently added users" actually useful!
Why
Right now it's screen space that is pretty much totally wasted - not informative and no utility
Details
1/ Add the email to the text. We have so many "Name Not Provided" that it's a bit ridiculous.
2/ Change the link to edit rather than show. (We have no user show in the controller, and, really, the superuser is not going to be signing in unless there's something that needs to be done!)
3/ Limit the view to the 20 most recent users
Criteria for completion
The text was updated successfully, but these errors were encountered: