mirror of
https://github.com/hoppscotch/hoppscotch.git
synced 2026-04-26 01:06:00 +03:00
[PR #4344] [MERGED] fix: preemptly update pwa settings to avoid missing files in the service worker cache #4762
Labels
No labels
CodeDay
a11y
browser limited
bug
bug fix
cli
core
critical
design
desktop
discussion
docker
documentation
duplicate
enterprise
feature
feature
fosshack
future
good first issue
hacktoberfest
help wanted
i18n
invalid
major
minor
need information
need testing
not applicable to hoppscotch
not reproducible
pull-request
question
refactor
resolved
sandbox
self-host
spam
stale
testmu
wip
wont fix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/hoppscotch#4762
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/hoppscotch/hoppscotch/pull/4344
Author: @amk-dev
Created: 9/13/2024
Status: ✅ Merged
Merged: 9/19/2024
Merged by: @AndrewBastin
Base:
patch← Head:fix/empty-screen-on-update📝 Commits (2)
c9b0e93chore: update vite-plugin-pwabd9b2e6chore: revert version bump and apply higher max cache file size📊 Changes
3 files changed (+132 additions, -33 deletions)
View changed files
📝
packages/hoppscotch-selfhost-web/package.json(+2 -2)📝
packages/hoppscotch-selfhost-web/vite.config.ts(+1 -1)📝
pnpm-lock.yaml(+129 -30)📄 Description
Recently we had the below issue on our cloud instance, this PR adds the necessery changes required to preemptively fix this.
Before
Whenever a release is out, our webapp seems to get an empty screen. this is happening due to a resource going above the max file size for getting cached and being left out from the precache.
so this is happening,
After
In our cloud instance, I've increased the max cache size to 10MB using
maximumFileSizeToCacheInBytesoption in workbox. but here i'm not doing that because it hasn't happened yet. but incase any file hits that limit. the build will fail and we'll know exactly what change causing it, and we can make whatever change is appropriate then.vite-plugin-pwa, it will throw an error instead of a warning if a file goes above the cachable limit. so we'll know before we ship it. so i've updated thevite-plugin-pwato the latest version.NB: although we have a cached app, and people might have good internet, we should look into factors affecting our bundle size as a lower priority thing.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.