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

Configuration for "peekable room" in single room mode #191

Open
ashfame opened this issue Feb 28, 2023 · 2 comments
Open

Configuration for "peekable room" in single room mode #191

ashfame opened this issue Feb 28, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@ashfame
Copy link
Member

ashfame commented Feb 28, 2023

On Matrix.org it can take a good while (1min+) at times to just get the room state (visibility=world_readable)

So, if we add a configuration along side single room mode (when room is specified), whether the room is peekable or not, we can skip checking for it dynamically and simply fetch messages.

This should speed up the time its required before the timeline can be rendered for the user.

@psrpinto
Copy link
Member

The problem I see with this is that the state of the room can change (e.g. history_visibility can become non-world-readable) after it has been configured.

@ashfame
Copy link
Member Author

ashfame commented Feb 28, 2023

Yeah! But in order to make that fast, its either caching and then turning out to be incorrect or user supplying that information. Latter seems better. And if we are unable to fetch messages, we can still fail gracefully.

@ashfame ashfame added the enhancement New feature or request label Mar 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants