-
Notifications
You must be signed in to change notification settings - Fork 404
Update implementation of MSC4306: Thread Subscriptions to include automatic subscription conflict prevention as introduced in later drafts. #18756
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
a3a8e02
Add a `AnyEventId` pydantic-validated type
reivilibre 73f653e
Add a test helper for sending a sequence of events
reivilibre 915e588
Add conflict prevention to automatic thread subscriptions
reivilibre 9c88178
Newsfile
reivilibre f3b7d5d
Update synapse/storage/databases/main/thread_subscriptions.py
reivilibre c6b8a4d
Update synapse/handlers/thread_subscriptions.py
reivilibre 956532a
Use new error codes from latest MSC4306 draft
reivilibre e3bea40
send_events -> send_messages
reivilibre 17ef8e3
Add event ID examples
reivilibre 954eb7a
Add `EventOrderings` dataclass
reivilibre c78840f
kwargify ints
reivilibre e90af84
Clamp stream_ordering to 0 instead of asserting
reivilibre 955e6dd
Rename automatic arg
reivilibre ab34932
Add case-by-case summary
reivilibre ae99ede
Unshadow `event`
reivilibre d61234d
Error wording tweak
reivilibre 99afc12
Fix test error codes
reivilibre de59b63
Remove unnecessary assertion
reivilibre a7b4ea5
Remove unnecessary clamping
reivilibre f56bdf9
linewrp
reivilibre 29c1ae4
Point to docstring for case-by-case
reivilibre fc05b0b
fixup! kwargify ints
reivilibre 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
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.