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

Delete users #92

Closed
tomlebl opened this issue May 24, 2024 · 4 comments · Fixed by #114
Closed

Delete users #92

tomlebl opened this issue May 24, 2024 · 4 comments · Fixed by #114
Assignees
Labels
enhancement New feature or request

Comments

@tomlebl
Copy link
Contributor

tomlebl commented May 24, 2024

We should enable deleting users if they have not acquired any experiments yet.
It would be done through checkboxes and action button. On backend we would check if a user has any experiments and if not then delete user and if yes we would make them inactive.

@tomlebl tomlebl added the enhancement New feature or request label May 24, 2024
@tomlebl tomlebl moved this to Todo in NOMAD system May 24, 2024
@tomlebl tomlebl assigned tomlebl and unassigned tomlebl Jun 7, 2024
@tomlebl
Copy link
Contributor Author

tomlebl commented Jun 7, 2024

@hunxjunedo
You could possibly have a look on this one. It would require to add checkboxes in user table and action button in /NavBar/PageHeader/UsersTabControls that would delete all checked users. You will need to check on backend if there are any experiments that belong to the user that is going to be deleted. If user has experiments than it does not get deleted but instead it becomes inactive {isActive: false}. Upon receiving response you could display message/modal.info showing how many users were deleted/made inactive.
This one is slightly harder than previous two but in the same corner of the app that you have become familiar with.

@tomlebl tomlebl assigned tomlebl and unassigned tomlebl Jun 7, 2024
@hunxjunedo
Copy link
Contributor

Assign this to me, let me get into this.

@tomlebl tomlebl moved this from Todo to In Progress in NOMAD system Jun 7, 2024
@hunxjunedo
Copy link
Contributor

Hey sorry for the delay, have been quite busy lately, coming with a PR today inshallah.

@tomlebl
Copy link
Contributor Author

tomlebl commented Jun 11, 2024

No rush, I have bee quite busy as well. Cheers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants