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

[UI] User management: Sorting feature #26170

Closed
pmaier1 opened this issue Sep 21, 2016 · 11 comments
Closed

[UI] User management: Sorting feature #26170

pmaier1 opened this issue Sep 21, 2016 · 11 comments

Comments

@pmaier1
Copy link
Contributor

pmaier1 commented Sep 21, 2016

Category: UI/admin
Quick summary use case: As an admin I want to be able to sort user accounts according to the criteria in the user interface (e.g. last login)
Problem it solves / Why I think this is important: Admins get a better overview on their userbase and have the ability to separate out users by sorting according to the criteria

I'm aware that this should probably be done via OCC commands for larger instances but I think it would be nice to have in the UI.

Core feature? Please share your thoughts on this!

@pmaier1
Copy link
Contributor Author

pmaier1 commented Sep 21, 2016

#26171

@PVince81
Copy link
Contributor

Best would be to do the sorting on the user backend, which means that we need a way to pass the sorting information to the user backend. User backend can be local users, LDAP or any others.
This, assuming that the user backend supports sorting...

Another idea is to sort on the client side (either on the PHP side after retrieving users from the user backend or on the JS side), but this is not compatible with pagination. So server-side sorting is likely the best approach.

@PVince81
Copy link
Contributor

Actually a duplicate of #12597.

Which one to close ?

@pmaier1
Copy link
Contributor Author

pmaier1 commented Oct 11, 2016

Well, the other issue is about sorting alphabetically. Here we want to sort according to the criteria in the user management view, e.g. group membership, last login. We should add sorting alphabetically here of course but I think this FR is more comprehensive.

@PVince81
Copy link
Contributor

Ah, I actually meant #12598

This is all from an earlier on-going task for improving the user management: #12998

@pmaier1
Copy link
Contributor Author

pmaier1 commented Oct 11, 2016

Yup, this is indeed identical :) I don't care which one to close, it should be pursued anyway.

@PVince81
Copy link
Contributor

Better idea: first implement the user account table #23558 and then just sort on that. This wouldn't require any interaction with any user backend nor require these to support sorting. We'd just use plain old database sorting on the user account table.

Would set to backlog for now.

@PVince81 PVince81 added this to the backlog milestone Jan 27, 2017
@ownclouders
Copy link
Contributor

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 7 days. Feel free to reopen this issue if you deem it appropriate.

(This is an automated comment from GitMate.io.

@PVince81
Copy link
Contributor

there are some challenges on the backend related to natural sort and pagination, we might need to rethink how our APIs work: #7244 (comment)

@PVince81
Copy link
Contributor

Let's change the approach:

  • server is doing the sorting and pagination
  • server operated on oc_accounts for now (if not already)
  • client re-fetches data on sort, does not do any sorting

@stale
Copy link

stale bot commented Sep 21, 2021

This issue has been automatically closed.

@stale stale bot closed this as completed Sep 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants