Skip to content
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

WIP: Support for msc3946 #139

Closed
wants to merge 4 commits into from
Closed

WIP: Support for msc3946 #139

wants to merge 4 commits into from

Conversation

DMRobertson
Copy link
Contributor

@DMRobertson DMRobertson commented May 31, 2023

Closes #38

Utterly WIP. I'm opening the PR for better visibility of in-flight work.

Concerns:

  • What is there to stop me from marking rooms A and B as both superseding room C? At present the latest seen predecessor event wins.
  • If I mark room A as superseding B1, then mark A as superseding B2, is B1 still considered to be replaced by A?
  • What do we do if we learn that A supersedes B but we have no knowledge of room B? It would be a shame if we later joined B and didn't present it as being replaced by B.

@DMRobertson
Copy link
Contributor Author

Going to drop this. I didn't get far, it's not a priority, and there looks to have been some bitrot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for MSC3946: Dynamic room predecessor
1 participant