-
-
Notifications
You must be signed in to change notification settings - Fork 605
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
Send MSC3981 'recurse' param on /relations
endpoint on Matrix 1.10 servers
#4023
base: develop
Are you sure you want to change the base?
Conversation
So we will send the 'recurse' parameter unprefixed for servers that support 1.10 (when it's released).
For links: this is about matrix-org/matrix-spec-proposals#3981. |
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.
Seems like you're not allowed to do this yet, because it will create problems if the MSC doesn't get written up by the time spec v1.10 is released.
Fair enough, I'll leave it as a draft until then. |
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.
Looks good.
Could do with a changelog entry though.
Changelog entry will be done by the release process from the PR title. |
Not if it's T-Task, I think? Also, "Mark MSC3981 as stable in Matrix 1.10" is a crap changelog entry. What does it actually mean, for a user of matrix-js-sdk? |
/relations
endpoint on Matrix 1.10 servers
Done, hopefully this is okay information-density wise. |
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.
thanks
So we will send the 'recurse' parameter unprefixed for servers that support 1.10 (which most should now do, since it's released).
Checklist
This change is marked as an internal change (Task), so will not be included in the changelog.