-
Notifications
You must be signed in to change notification settings - Fork 379
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
MSC:1819 Remove Presence Lists #1819
Conversation
In the interest of getting this in for r0 (and because it looks great to me - thank you!): @mscbot fcp merge |
Team member @turt2live has proposed to merge this. The next step is review by the rest of the tagged people: No concerns currently listed. Once a majority of reviewers approve (and none object), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up! See this document for info about what commands tagged team members can give me. |
🔔 This is now entering its final comment period, as per the review above. 🔔 |
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.
Please can we be clear exactly what we are removing from the spec? I assume it's POST /_matrix/client/r0/presence/list/{userId}
and GET /_matrix/client/r0/presence/list/{userId}
?
(as a matter of preference for future reference: please wrap docs at 80 characters so that they are easier to review)
|
||
The proposal is to simply remove references to presence lists with a view to revisiting the same ideas in the future. | ||
|
||
This MSC addresses [#1810](https://github.com/matrix-org/matrix-doc/issues/1810) |
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.
This MSC addresses [#1810](https://github.com/matrix-org/matrix-doc/issues/1810) | |
This MSC addresses [#1810](https://github.com/matrix-org/matrix-doc/issues/1810). |
Co-Authored-By: neilisfragile <[email protected]>
Add references to exactly what this proposal would remove
A miscommunication and subsequent timing error on our part has lead to this MSC's FCP not scheduled to clear in time. Given the extensive opportunity for people to review, and our goal to cut a s2s r0 today, we are fast forwarding this FCP to end now. There have been no requested changes while this has been in FCP or before this proposal was even opened. |
This proposal has also jumped to merged because the spec PR has been reviewed and merged while this has been in FCP. No changes have been made to the MSC, and as such no further changes are required to the spec for this MSC. |
Rendered