-
Notifications
You must be signed in to change notification settings - Fork 9
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
#3546 - Added event api endpoint caching #3549
base: main
Are you sure you want to change the base?
Conversation
Using npm stale-while-revalidate-cache package
Coverage report
Test suite run success13 tests passing in 1 suite. Report generated by 🧪jest coverage report action from 3bacc45 |
Deployed changes to https://app-sswwebsite-9eb3-pr-3549.azurewebsites.net |
PageSpeed Insights
View the full report https://pagespeed.web.dev/report?url=https://app-sswwebsite-9eb3-pr-3549.azurewebsites.net |
- Using `lru-cache` to limit memory growth - Limiting cache key combinations to avoid key polluting - Max length of presenter to avoid large key sizes - LRU cache expires after 1 hour
Deployed changes to https://app-sswwebsite-9eb3-pr-3549.azurewebsites.net |
PageSpeed Insights
View the full report https://pagespeed.web.dev/report?url=https://app-sswwebsite-9eb3-pr-3549.azurewebsites.net |
Using npm
stale-while-revalidate-cache
packageAffected routes:
Fixed 🐛 Bug - Events API is slow to return Past and Upcoming events #3546
If adding a new page, I have followed the 📃 New Webpage issue template
Include done video or screenshots
Figure: Caching performance on endpoint (local dev)