-
-
Notifications
You must be signed in to change notification settings - Fork 101
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
Spec omit_members
for /v2/send_join
#1393
Conversation
we are extending v2, but not v1.
I haven't written much about how this is meant to be used, since faster-joins are still somewhat in development, and that's really the topic of MSC3902. It's really just meant to lay down the mechanism on which faster-joins are based. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
I haven't made any effort to maintain backwards compatibility (unlike in matrix-org/synapse#14832). I'm happy to do so if needed, but might need some guidance. Synapse issue: matrix-org/synapse#14828 Synapse PR: matrix-org/synapse#14832 Complement PR: matrix-org/complement#583 GMSL PR: #350 MSC: matrix-org/matrix-spec-proposals#3706 Spec diff: matrix-org/matrix-spec#1393 ### Pull Request Checklist * [x] Pull request includes a [sign off](https://github.com/matrix-org/dendrite/blob/master/docs/CONTRIBUTING.md#sign-off) Signed-off-by: `David Robertson <davidr@element.io>`
Per MSC3706
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
did you forget to change the example of the sendJoinV1 response?
The event
should be gone, I suppose.
Per MSC3706
Preview: https://pr1393--matrix-spec-previews.netlify.app