MSC4132: Deprecate Linking to an Event Against a Room Alias

[pixlwave]

Rendered

FCP tickyboxes

[turt2live]

... and because I haven't sent enough comments in the last few minutes: the deprecation requirements can be clarified concurrent to checkbox gathering, imo. I see no reason to delay this MSC.

@mscbot fcp merge

[mscbot]

Team member @mscbot has proposed to merge this. The next step is review by the rest of the tagged people:

• [x] @clokep
• [x] @dbkr
• [x] @uhoreg
• [x] @turt2live
• [x] @ara4n
• [ ] @anoadragon453
• [ ] @richvdh
• [x] @tulir
• [x] @erikjohnston
• [x] @KitsuneRal

Concerns:

• outstanding comments

Once at least 75% of reviewers approve (and there are no outstanding concerns), 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 information about what commands tagged team members can give me.

[pixlwave]

... and because I haven't sent enough comments in the last few minutes: the deprecation requirements can be clarified concurrent to checkbox gathering, imo. I see no reason to delay this MSC.

Just for clarification as I haven't done this before, I think this is saying its fine to update the proposal based on the comments whilst SCT members are looking at it?

[pixlwave]

Updated taking all of the comments above into consideration.

[turt2live]

... and because I haven't sent enough comments in the last few minutes: the deprecation requirements can be clarified concurrent to checkbox gathering, imo. I see no reason to delay this MSC.

Just for clarification as I haven't done this before, I think this is saying its fine to update the proposal based on the comments whilst SCT members are looking at it?

Yes, sorry. The proposal can be updated in all kinds of ways up until Final Comment Period (FCP) actually starts - when 8/10 people check their boxes and no concerns are formally raised. Things which SCT members believe should prevent acceptance of the MSC are raised via @mscbot concern. Everything else is non-blocking, though may still feel important to address.

[turt2live]

(though please avoid force pushing, as it makes review harder. We squash all the commits down to a single commit upon acceptance)

[pixlwave]

My bad, good to know you squash 👍

[dbkr]

@mscbot concern outstanding comments

[dbkr]

@mscbot resolve outstanding comments

[mscbot]

:bell: This is now entering its final comment period, as per the review above. :bell:

[mscbot]

The final comment period, with a disposition to merge, as per the review above, is now complete.

[turt2live]

Spec PR: https://github.com/matrix-org/matrix-spec/pull/1823

[ara4n]

ironically, i just found a permalink indexed by room alias in the wild: https://github.com/element-hq/element-web/issues/2150#issue-176068929

[richvdh]

merged! :tada: