Add API endpoint to delete org #1448
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tw4l
force-pushed
the
issue-890-org-export-import
branch
from
05d3c4e
to
5a412c0
Compare
tw4l
force-pushed
the
issue-903-delete-org
branch
2 times, most recently
from
70546a8
to
64c6ad6
Compare
|
Dependent on #1394 |
tw4l
force-pushed
the
issue-890-org-export-import
branch
3 times, most recently
from
4d5d002
to
45909cc
Compare
tw4l
force-pushed
the
issue-890-org-export-import
branch
from
d6ab28a
to
618b9c0
Compare
tw4l
force-pushed
the
issue-903-delete-org
branch
from
64c6ad6
to
8bb8c76
Compare
ikreymer
requested changes
Jul 2, 2024
There was a problem hiding this comment.
For all of the to_list() calls, we should probably use a while loop as shown here: https://motor.readthedocs.io/en/stable/api-asyncio/cursors.html#motor.motor_asyncio.AsyncIOMotorCursor.to_list which allows for iterating in N results, otherwise looks good!
ikreymer
reviewed
Jul 3, 2024
ikreymer
reviewed
Jul 3, 2024
tw4l
force-pushed
the
issue-903-delete-org
branch
from
0c7447a
to
74a207f
Compare
Also deletes users if they only belong to this org.
tw4l
force-pushed
the
issue-903-delete-org
branch
from
74a207f
to
dddcb5c
Compare
tw4l
force-pushed
the
issue-903-delete-org
branch
from
dddcb5c
to
2fa86f0
Compare
|
@ikreymer Latest change re: replicas tested and working as expected, thanks for that! |
ikreymer
approved these changes
Jul 3, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #903
Adds superuser-only API endpoint to delete an org and all of its data