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

fix: Add matchOptions to runtimeCaching.options.expiration scheme #2865

Closed
wants to merge 1 commit into from

Conversation

cawa-93
Copy link

@cawa-93 cawa-93 commented Jun 12, 2021

Prior to filing a PR, please:

  • open an issue to discuss your proposed change.
  • ensure that gulp build && gulp lint test passes locally.

R: @jeffposnick @tropicadri

Fixes #2864

Description of what's changed/fixed.
Add matchOptions to runtimeCaching.options.expiration scheme

@jeffposnick
Copy link
Contributor

Thanks for the contribution—unfortunately, we've got a very large PR out for review now that migrates workbox-build to TypeScript. As part of this migration, the way options are set and validated has changed.

This specific PR won't work with the planned changes, but I'll leave the underlying issue open. If you want to submit a new PR against the TypeScript codebase once #2867 is merged, you're welcome to—otherwise, someone else from the project can make that change.

@cawa-93 cawa-93 deleted the expiration-scheme branch June 15, 2021 17:50
@cawa-93 cawa-93 restored the expiration-scheme branch June 17, 2021 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

There is no way to set matchOptions for ExpirationPlugin in generateSW
2 participants