Skip to content
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

Replace to workbox-webpack-plugin from offline-plugin #18409

Merged
merged 1 commit into from Aug 25, 2022
Merged

Replace to workbox-webpack-plugin from offline-plugin #18409

merged 1 commit into from Aug 25, 2022

Conversation

ykzts
Copy link
Sponsor Member

@ykzts ykzts commented May 13, 2022

Separated from #18369.

The offline-plugin we use for Service Worker has not been maintained for many years (3 years since the last release) and does not support webpack v5. Therefore, it is not possible to migrate to webpack v5 or to Shakapacker, which depends on webpack v5. Replace offline-plugin with workbox-webpack-plugin which supports both webpack v5 and webpack v4 for better compatibility.

@ykzts ykzts added dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code labels May 13, 2022
@ykzts
Copy link
Sponsor Member Author

ykzts commented Jun 28, 2022

Conflicts resolved.

@ykzts
Copy link
Sponsor Member Author

ykzts commented Aug 25, 2022

Conflicts resolved.

@Gargron Gargron merged commit 81e1cc5 into main Aug 25, 2022
@Gargron Gargron deleted the workbox branch August 25, 2022 18:10
kadoshita pushed a commit to kadoshita/mastodon that referenced this pull request Nov 19, 2022
ClearlyClaire pushed a commit to ClearlyClaire/mastodon that referenced this pull request Jul 6, 2023
registerRoute(
({ request }) => ['audio', 'image', 'track', 'video'].includes(request.destination),
new CacheFirst({
cacheName: `m${CACHE_NAME_PREFIX}media`,
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder why 'm' is prefixed to CACHE_NAME_PREFIX.
My guess it's a mistake from when typing in media?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants