-
Notifications
You must be signed in to change notification settings - Fork 437
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
Update CARE prompt when new version available #4298
Update CARE prompt when new version available #4298
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@gigincg regd. if |
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.
LGTM, great stuff! 🚀
Maybe we can provide an option to snooze updates. Snoozing will make it popup every 5 hours or so. User can then manually update the software in the user account page. What say?
👋 Hi, @rithviknishad, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
SonarCloud Quality Gate failed. 0 Bugs No Coverage information |
LGTM |
Proposed Changes
ButtonV2
in unauthenticated pages #4297Technical changes:
/build-meta.json
generation flow./build-meta.json
file and compares it with theapp-version
stored inlocalStorage
. If both versions mismatch (or even ifapp-version
does not exist inlocalStorage
)hasUpdate
is set totrue
, which shows the popup.updateApp
is invoked by button onClick, it empties the cache and invokeswindow.locations.reload
, and sets two items in localStorage:app-updated
is cleared.app-updated
key is used just to show the update success popup when the app has been updated after the reload.2022-12-13.16-45-26.mp4
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist