-
Notifications
You must be signed in to change notification settings - Fork 258
WS-1396: Lite site caching PoC #13255
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
base: latest
Are you sure you want to change the base?
WS-1396: Lite site caching PoC #13255
Conversation
…rolled format string Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
andrewscfc
left a comment
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.
few questions to aid my understanding 👍
| return response; | ||
| })(), | ||
| ); | ||
| } else if (isRequestForMostRead) { |
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.
where is this variable defined? I can't find it anywhere on the diff?
| )}`; | ||
| const HOME_PAGE_URL = `${getEnvConfig().SIMORGH_BASE_URL}/${service}`; | ||
|
|
||
| const fetchMostReadItems = async () => { |
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.
why can't this request be made from within the service worker?
| const cache = await caches.open(cacheName); | ||
| if (hasOfflinePageFunctionality) { | ||
| await cache.add(OFFLINE_PAGE); | ||
| await cacheMostReadStories(cache); |
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.
where is this function defined?
A draft PoC for JIRA: WS-1396 SPIKE: Lite site homepage caching in offline mode
Summary
A PR to showcase lite site homepage and most read articles caching capabilities.
Code changes
Developer Checklist
Testing
Ready-For-Test, Local)Ready-For-Test, Test)Ready-For-Test, Preview)Ready-For-Test, Live)Additional Testing Steps
Useful Links