This repository was archived by the owner on Apr 26, 2024. It is now read-only.
-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Convert delete room admin API to async endpoint #11223
Merged
Merged
Changes from 5 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
c35a367
Convert delete room admin API to async endpoint
dklimpel 69bba24
newsfile
dklimpel a9f9e3a
disable flaky test
dklimpel 9d381cf
Merge branch 'develop' into delete_room_bg
dklimpel 6ddcc8e
Fix wrong unit test `test_block_room_and_not_purge`
dklimpel cf8b612
Apply suggestions from code review
dklimpel a37b09c
update docstrings and mark private fields with `_`
dklimpel 5b24839
Merge remote-tracking branch 'synapse/develop' into delete_room_bg
dklimpel 3f9a4f4
Reuse `RoomShutdownHandler.shutdown_room`
dklimpel a482170
Merge remote-tracking branch 'synapse/develop' into delete_room_bg
dklimpel 2f1f483
try flaky unit test
dklimpel 60ef11e
rework request of purge status
dklimpel 76a62d5
add error message if delete fails
dklimpel 7de0bf1
Apply suggestions from code review
dklimpel a618958
add simple unit test for `/purge_history_status`
dklimpel 4f42348
rename response dicts
dklimpel 5a764c8
add new status API `/rooms/delete_status/<purge_id>`
dklimpel 32c0694
update doc
dklimpel 7821ed2
update docstring
dklimpel fb2019d
Merge remote-tracking branch 'synapse/develop' into delete_room_bg
dklimpel 3b3cf2a
fix merge
dklimpel bd70ffd
update test response code, introduced in #11228
dklimpel 383d808
Merge remote-tracking branch 'synapse/develop' into delete_room_bg
dklimpel 05dd106
make compatible with #11228
dklimpel 11d9e87
rename status
dklimpel fea716f
separate `purge` and `delete`
dklimpel 1e5c5a4
Apply suggestions from code review
dklimpel 949e55f
lint and small fixes
dklimpel File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| Add a new version of delete room admin API `DELETE /_synapse/admin/v2/rooms/<room_id>` to run it in background. Contributed by @dklimpel. |
This file contains hidden or 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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.