-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
feat: Split display name and username columns #45245
Conversation
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.
Makes sense for me, lets wait for designers.
Might be some intersection with #34178 (to avoid future code debt) though I didn't really follow where that stands since it's merged and the menu entry to get to Accounts is still labeled as Users so dunno. In any case, seems header Username -> Login I think? |
I think this was intentional @skjnldsv ?
On the login page it is also "username" so I will keep it the same |
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.
Seems good!
Agree with @joshtrichards that Username should be "Account name" (same on the login screen). But it’s something for a separate pull request. :)
Also question @Pytal – isn’t the sorting we add by "Last login" only? (And display name which is the default.) Or are we just going to make everything sortable? |
Sure, can do in a followup :)
Will just be displayname, userId, and last login for now as that is what has been implemented on the backend, not sure about others as it may be complicated |
/compile amend / |
98fd89b
to
3e53a80
Compare
Signed-off-by: Christopher Ng <[email protected]>
Signed-off-by: Christopher Ng <[email protected]> Signed-off-by: nextcloud-command <[email protected]>
3e53a80
to
67dcd33
Compare
Summary
Checklist