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

Cockpit: sort users and groups #5969

Closed
federicoballarini opened this issue Nov 29, 2019 · 6 comments
Closed

Cockpit: sort users and groups #5969

federicoballarini opened this issue Nov 29, 2019 · 6 comments
Labels

Comments

@federicoballarini
Copy link

@federicoballarini federicoballarini commented Nov 29, 2019

Can be helpful to sort users and groups list by alphabetical order.

Proposed solution

Sort current list without change view.

Alternative solutions

Change view using a table.

See also

https://community.nethserver.org/t/sorted-users-and-groups/


/cc @edospadoni

@DavidePrincipi DavidePrincipi added this to ✋ Needs review in NethServer 7 via automation Nov 29, 2019
@federicoballarini

This comment has been minimized.

Copy link
Author

@federicoballarini federicoballarini commented Dec 1, 2019

@edospadoni Could you give me some hints on how do you want to solve it? Thanks! :)

@edospadoni

This comment has been minimized.

Copy link
Member

@edospadoni edospadoni commented Dec 2, 2019

It's a normal object sorting by value on javascript object. When the Cockpit API returns the users object, sort it.

edospadoni added a commit to NethServer/nethserver-cockpit that referenced this issue Dec 4, 2019
@edospadoni

This comment has been minimized.

Copy link
Member

@edospadoni edospadoni commented Dec 4, 2019

Test case:

  • check users and groups list are alphabetically sorted
@edospadoni edospadoni added the testing label Dec 4, 2019
@nethbot

This comment has been minimized.

Copy link
Member

@nethbot nethbot commented Dec 4, 2019

in 7.7.1908/testing:

@gsanchietti

This comment has been minimized.

Copy link
Member

@gsanchietti gsanchietti commented Dec 4, 2019

Tested by Mark, verified.

@gsanchietti gsanchietti added verified and removed testing labels Dec 4, 2019
@nethbot

This comment has been minimized.

Copy link
Member

@nethbot nethbot commented Dec 4, 2019

in 7.7.1908/updates:

@gsanchietti gsanchietti closed this Dec 4, 2019
NethServer 7 automation moved this from ✋ Needs review to 🗑 Done Dec 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
NethServer 7
🗑 Done
4 participants
You can’t perform that action at this time.