-
-
Notifications
You must be signed in to change notification settings - Fork 97
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
Document MSC3758, MSC3966, MSC3873, and MSC3980. #1464
Document MSC3758, MSC3966, MSC3873, and MSC3980. #1464
Conversation
8ed2b79
to
4cd7faf
Compare
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 looks great, and I appreciate your clear wording.
A few bits and bobs to fix up, and a leading question.
Co-authored-by: Andrew Morgan <1342360+anoadragon453@users.noreply.github.com>
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.
These changes LGTM. Thanks!
/me performs ceremony to merge |
* Add information on MSC3758: event_property_is. * Add information on MSC3966: event_property_contains. * Add information on MSC3873 dotted-path escape rules. * Newsfragment * Update sync filter with ref to appendix. * Escape example key. * Fix typos. Co-authored-by: Andrew Morgan <1342360+anoadragon453@users.noreply.github.com> * Fix links. * Clarify the appendix a bit. * Clarify support values. * Add MSC3980 to changelog. --------- Co-authored-by: Andrew Morgan <1342360+anoadragon453@users.noreply.github.com>
This documents the new push rule conditions:
event_property_is
(MSC3758)event_property_contains
(MSC3966)As well as the dotted-path escape rules from MSC3873 and MSC3980.
Based on #1461.Preview: https://pr1464--matrix-spec-previews.netlify.app