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

Proof of concept for MSC3613: Combinatorial join rules#11668

Closed
turt2live wants to merge 9 commits intodevelopfrom
travis/experiment/join_rules
Closed

Proof of concept for MSC3613: Combinatorial join rules#11668
turt2live wants to merge 9 commits intodevelopfrom
travis/experiment/join_rules

Conversation

@turt2live
Copy link
Copy Markdown
Member

@turt2live turt2live commented Dec 31, 2021

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

This was originally built to prove the concept, and is contained in an unstable room version to help test it. The goal is to get this landed so people in the wild can experiment with it (and build clients against it).

The code quality of this PR is potentially incorrect - I'm hoping code review will cover anything which isn't great. As a not-python developer, I'm not going to be too hurt to hear that it's ugly ;)

TODO:

  • Tests

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)

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

This is not a comprehensive implementation and was only built to prove that the idea works in a limited test case (knocking+restricted). 

The following still need to be considered/completed:
* [ ] Make code quality on par with the rest of the project
* [ ] Tests
* [ ] Use the utility function in all the places
* [ ] Update redaction algorithm
* [ ] Remove auth rule changes (see MSC)
* [ ] Fix handling of `allow` (see TODOs in code)
@turt2live turt2live added the Z-Help-Wanted We know exactly how to fix this issue, and would be grateful for any contribution label Dec 31, 2021
@turt2live
Copy link
Copy Markdown
Member Author

MSC rejected

@turt2live turt2live closed this May 4, 2022
@turt2live turt2live deleted the travis/experiment/join_rules branch May 4, 2022 02:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Z-Help-Wanted We know exactly how to fix this issue, and would be grateful for any contribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants