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

Redesign user index #1333

Closed
janno42 opened this issue Apr 8, 2019 · 7 comments · Fixed by #1667
Closed

Redesign user index #1333

janno42 opened this issue Apr 8, 2019 · 7 comments · Fixed by #1667
Assignees
Labels
[C] Backend Focuses on backend implementation [C] Frontend Focuses on frontend implementation [P] Medium Medium priority [S] Easy This issue is self-contained and does not touch too many parts. Good choice for beginners. [T] Refactoring Existing parts should become faster, more readable, or in any other way better.
Milestone

Comments

@janno42
Copy link
Member

janno42 commented Apr 8, 2019

The user index page loads quite slowly. This could probably be improved be removing badges that are not really needed ("Contributor", "Responsible", "External", "No email") and postponing the can_be_deleted_by_manager check to be run only after clicking the button and returning an error message if not possible
It should be investigated if further improvements to the loading time are possible.

@janno42 janno42 added [C] Backend Focuses on backend implementation [C] Frontend Focuses on frontend implementation [P] Minor Minor priority [T] Refactoring Existing parts should become faster, more readable, or in any other way better. labels Apr 8, 2019
@karyon
Copy link
Collaborator

karyon commented Mar 14, 2020

In an offline discussion, @janno42 realized that the delete button could simply be moved to the user edit form.

@janno42
Copy link
Member Author

janno42 commented Oct 7, 2020

(edited, see discussion below)

In most cases we don't need a complete list of all users at all. So the user index page should be changed to a new overview page like the staff index giving access to:

  • user import
  • user adding
  • user merge
  • user bulk update
  • the complete user list as it is now but extended by filters for proxy users, managers, grade publishers, and reviewers
  • a new search field where all users (including inactive and proxy users) can be found via a user selection field which then allows to open the respective user's edit form for a specific user

And in addition:

  • the user edit form needs to get a delete button as stated above

@janno42 janno42 added [P] Medium Medium priority [S] Easy This issue is self-contained and does not touch too many parts. Good choice for beginners. and removed [P] Minor Minor priority labels Oct 7, 2020
@karyon
Copy link
Collaborator

karyon commented Oct 7, 2020

to avoid any confusion, you want to keep the full list but extend it with more filters, and add buttons to import/merge etc at the top?

and your last bullet point I didn't understand. What does it do differently than the current search field?

@janno42
Copy link
Member Author

janno42 commented Oct 7, 2020

the full list with additional filters is for the rare cases where you might want to see all reviewers when the student representative group changes or something like that. but it needs to be an extra page instead of the index, so that for all other use cases the loading time of the list doesn't matter anymore.

the search field should be placed on the (new) user index page and could work just like the contributor selection fields but with a button next to it opening the selected profile.

@janno42 janno42 changed the title Improve user index loading time Redesign user index Oct 7, 2020
@karyon
Copy link
Collaborator

karyon commented Oct 7, 2020

ooooh i see. So it's a completely new page with a button leading to the old one.

i would suggest leaving out the additional filters to the old page for the first PR to make this issue more accessible. maybe add the delete button to the user edit form instead, so the new page can cover more use cases. maybe the old page really becomes irrelevant then (or irrelevant enough that typing 'reviewer' into the search field is sufficient for your "see all reviewers" use case)

@janno42
Copy link
Member Author

janno42 commented Oct 7, 2020

ah right, i had searching like that in mind and somehow forgot it on the way writing the requirement above. you are right: additional filters are not required. i'll update the list above and add the delete button thing.

@richardebeling richardebeling assigned lill28 and unassigned conradhalle Dec 14, 2020
@janno42 janno42 added this to the Summer 2021 milestone Jan 4, 2021
@janno42 janno42 modified the milestones: Summer 2021, Winter 2021 Jun 16, 2021
@janno42
Copy link
Member Author

janno42 commented Nov 1, 2021

@lill28 do you still want to work on this issue?

@richardebeling richardebeling linked a pull request Nov 17, 2021 that will close this issue
@janno42 janno42 modified the milestones: Winter 2021, Summer 2022 Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C] Backend Focuses on backend implementation [C] Frontend Focuses on frontend implementation [P] Medium Medium priority [S] Easy This issue is self-contained and does not touch too many parts. Good choice for beginners. [T] Refactoring Existing parts should become faster, more readable, or in any other way better.
Development

Successfully merging a pull request may close this issue.

5 participants