Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Add experimental MSC3604-opt1 room version #11662

Closed
wants to merge 1 commit into from

Conversation

turt2live
Copy link
Member

@turt2live turt2live commented Dec 29, 2021

Just need a place to store this.

MSC: matrix-org/matrix-spec-proposals#3604

Pull Request Checklist

  • Pull request is based on the develop branch
  • Pull request includes a changelog file. The entry should:
    • Be a short description of your change which makes sense to users. "Fixed a bug that prevented receiving messages from other servers." instead of "Moved X method from EventStore to EventWorkerStore.".
    • Use markdown where necessary, mostly for code blocks.
    • End with either a period (.) or an exclamation mark (!).
    • Start with a capital letter.
  • Pull request includes a sign off
  • Code style is correct
    (run the linters)

@clokep
Copy link
Member

clokep commented Dec 29, 2021

Is this different from the org.matrix.msc2176 room version that is already implemented? It looks like it builds on v9 instead of v6? Could we just update the other one (or remove it?) Adding separate ones for each potential v10 MSC seems tedious.

@turt2live
Copy link
Member Author

It builds off of v9, yea. Changing the existing one would mean changing how the room version operates, which is unsafe for rooms which might already be using it.

This is a draft and unlikely to end up in review. It's a place to point to in order to collect specific feedback and organize testing at the moment, not a candidate for inclusion in the actual project. This might change in the future though.

@reivilibre reivilibre added the T-Other Questions, user support, anything else. label Jan 6, 2022
@turt2live
Copy link
Member Author

(closing as not viable for MSC purposes)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Other Questions, user support, anything else.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants