-
Notifications
You must be signed in to change notification settings - Fork 260
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
Add a database cache for folders #1956
Conversation
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
c67e0bd
to
f40aba3
Compare
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very clean. I like it.
Also a test run was very promising and smooth. Keep them coming.
This is brilliant! Makes the app so much more usable. Thanks @ChristophWurst |
Thank you @PriceChild! I really appreciate the feedback! |
We also appreciate ratings on https://apps.nextcloud.com/apps/mail :) |
This drastically* speeds up page loads, especially with multi account setups.
Todo
*on my two-account test it took an avg of 8s to load a full page (showing a message as well) without this cache and an avg of 4.6 with.