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

introduce getSpaceHierarchy cache #1876

Open
WilsonLe opened this issue Jul 12, 2024 · 1 comment
Open

introduce getSpaceHierarchy cache #1876

WilsonLe opened this issue Jul 12, 2024 · 1 comment

Comments

@WilsonLe
Copy link

WilsonLe commented Jul 12, 2024

Describe your problem

With apps that call Api.getSpaceHierarchy often, it would be beneficial to have a built in cache of room hierarchy.

Describe your ideal solution

Introduce room hierarchy box in database cache

@WilsonLe WilsonLe changed the title Introduce Room Registry class with aggregated rooms, similar to Events in Timeline class introduce getSpaceHierarchy cache Jul 18, 2024
@krille-chan
Copy link
Contributor

Actually I'm not sure. The hierarchy can also be cached by the app. When you use this endpoint the consumer would just expect to get the newest version. Such complexity with caching could be added to dozens of other endpoints as well which would blow up the sdk too much. But that's only my opinion. What do the others think? @famedly/instant-messaging

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

No branches or pull requests

2 participants