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

[Feature] Bulk delete users #871

Merged
merged 24 commits into from Apr 5, 2023
Merged

[Feature] Bulk delete users #871

merged 24 commits into from Apr 5, 2023

Conversation

noliveleger
Copy link
Contributor

@noliveleger noliveleger commented Mar 22, 2023

Description

Allow the service account to delete users entirely

Related issues

kobotoolbox/kpi#4373

Copy link
Member

@jnm jnm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Initial read-through while still in progress. Looking good 👍

onadata/apps/logger/models/xform.py Outdated Show resolved Hide resolved
Comment on lines 56 to 57
def get_queryset(self):
return super(models.Manager, self).get_queryset()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pass 😊

@noliveleger noliveleger force-pushed the trash-bin branch 2 times, most recently from b1a9c32 to 61cc73f Compare March 23, 2023 18:39
@noliveleger noliveleger marked this pull request as ready for review March 31, 2023 15:00
@noliveleger noliveleger changed the title WIP - Trash bin [Feature] Bulk delete users Mar 31, 2023
onadata/apps/api/tests/viewsets/test_xform_viewset.py Outdated Show resolved Hide resolved
onadata/apps/api/viewsets/xform_viewset.py Outdated Show resolved Hide resolved
@jnm jnm merged commit 35a9850 into release/2.023.12 Apr 5, 2023
1 check passed
@jnm jnm deleted the trash-bin branch April 5, 2023 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants