-
Notifications
You must be signed in to change notification settings - Fork 507
There is no way to sort for users in the list of users #3265
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
Comments
Hi, first-time contributor. Can I try to work on this issue? |
Thank you for getting in touch. |
@rowasc any particular sort methods for the users you wanted? The only ones I can see at the moment would be alphabetical and roles. I was thinking of adding a button, but reading the contributor thing I found the platform-pattern-library you guys have and will try to use the search and filter one that is listed |
@c3ho Yes, alphabetical and roles would be a great start! Try with the search and filter-ui (its used it the map and data-view, you can peak at the code for that here: https://github.com/ushahidi/platform-client/tree/develop/app/main/posts/views/filters and more specifically here: https://github.com/ushahidi/platform-client/blob/develop/app/main/posts/views/filters/filters-dropdown.html). Thanks for looking into this 🎉 🎉 |
@rowasc @Angamanga Can I also work on this issue? |
@sparshkrjha Yes please go ahead :) Thank you for your interest in contributing! |
@sparshkrjha How is it going? Please let me know if you need any guidance with this! |
@Angamanga @rowasc Can I work on this issue? |
@himil-vasava Yes, please go ahead :) Thank you! |
@Angamanga should i create a dropdown for sorting and have two options in it - alphabetical and roles ? |
@himil-vasava Yes that sounds good 👍 |
@Angamanga could you please elaborate on sorting according to roles. |
@himil-vasava Roles is hard to sort, but we can filter them with checkboxes, i.e. if you select a specific role, only those users should be visible. Something like this: |
@Angamanga I think that feature is already there |
@himil-vasava Ah, just add alphabetical sort then :) |
@Angamanga could you please help me by telling what i am doing wrong here in the pull request |
Uh oh!
There was an error while loading. Please reload this page.
Want to work on this issue?🙋
Comment below. If someone else has claimed it, ask if they've opened a pull request already and if they're stuck -- maybe you can help them solve a problem or move it along!
Describe the bug
There is no way to sort users in the user list
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Users should be sortable.
Context
I think this may not be a bug, but i'm reporting it because it was part of the original regressions
The text was updated successfully, but these errors were encountered: