MSC3644: Extensible Events: Edits and replies #3644
Conversation
turt2live
changed the title
turt2live
added
kind:core
| A downside of this system is that relation-requiring events, such as poll responses and reactions, | ||
| are not editable because the relation will be overridden. Other proposals, like | ||
| [MSC3051](https://github.com/matrix-org/matrix-doc/pull/3051), aim to change the relation structure | ||
| wholesale to account for these cases. For the purposes of extensible events, and this proposal, the | ||
| intention is that these relation-requiring events describe overload mechanics on their own. For | ||
| example, polls only take into consideration the most recent event while reactions effectively rely | ||
| upon redact & re-send approaches. |
There was a problem hiding this comment.
Another affected system is MSC3440: Threads. It defines an m.thread relation which would be overridden by edits here. It's somewhat questionable if removing m.new_content is the correct approach, but for the moment I'm eager to punt the relation conflict out to other MSCs like the one referenced in the proposal text.
| external to this proposal. | ||
|
|
||
| Also as described, events can effectively change type under this system. This could have consequences | ||
| on systems like the [key verification framework](https://spec.matrix.org/v1.1/client-server-api/#key-verification-framework), |
There was a problem hiding this comment.
Eeeer, would this be an issue? Edits are about how to present an event to the user, the key verification says exactly which event type it needs when; it simply does not allow room for edits.
There was a problem hiding this comment.
Well, clients should take care to not support edits on those events by accident :3
There was a problem hiding this comment.
"could" is the operating keyword - I haven't really looked into it beyond realizing it's a potential area where we don't want edits. To be explored in a future MSC, though.
Rendered
Dependencies
This MSC requires the following to pass (or likely pass) FCP before being able to be FCP'd itself:
Preview: https://pr3644--matrix-org-previews.netlify.app