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

Provide a way to remove multiple users at the same time #217

Closed
llaske opened this issue Jan 18, 2020 · 6 comments
Closed

Provide a way to remove multiple users at the same time #217

llaske opened this issue Jan 18, 2020 · 6 comments

Comments

@llaske
Copy link
Owner

llaske commented Jan 18, 2020

Currently if an admin want to remove a bunch of users, he has no choice to remove them one by one using the dashboard. It could be a very long process if the number of users is large.
I suggest to add a way, for example using a specific syntax in the import script, to remove multiple users at the same time.

@NikhilM98
Copy link
Collaborator

Can we add something like: node scripts/remove_users.js filename.csv? The CSV will contain the name of the users the admin wants to delete.
It'll generate an output file with info about the users deleted/failed to delete.

@llaske
Copy link
Owner Author

llaske commented Jan 18, 2020

Will be nice if we could call it from the dashboard. It's why my first thought was to update the import script.

@NikhilM98
Copy link
Collaborator

@llaske if you want the bulk delete feature to be accessible from the dashboard then maybe we can modify the Import Users feature on the dashboard such that if we upload a CSV with user type as delete (Current user types are Admin, Student, and Teacher) then the script will find and delete that user.

@llaske
Copy link
Owner Author

llaske commented Jan 19, 2020

Sounds like a good idea but we need to update the seed_users.js script to do the same. So admin could do the action both from the command line and from the UI.

@NikhilM98
Copy link
Collaborator

Yes. I've already started working on seed_users.js before working on the API. I'll try to make a PR for both the scripts by tomorrow.

@llaske
Copy link
Owner Author

llaske commented Jan 25, 2020

Fixed in #218

@llaske llaske added this to the 1.3.0 milestone Oct 7, 2020
@llaske llaske closed this as completed Oct 11, 2020
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

2 participants