-
Notifications
You must be signed in to change notification settings - Fork 379
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
JsonManifestStrategy, support for FormatExtensionResolver #1588
JsonManifestStrategy, support for FormatExtensionResolver #1588
Conversation
…ub.com/skepp/LiipImagineBundle into json_manifest_version_strategy_support
thanks a lot for picking this up. there are conflicts (and the diff shows the changes already in #1529). can you squash the commits and rebase on json_manifest_version_strategy_support and solve conflicts? then it will be visible what actually changes in this pull request. |
…manifest_version_strategy_support
e1f15c4
to
b5b5569
Compare
Thank you, done |
4c911b2
into
liip:json_manifest_version_strategy_support
Apologies, this got very stale.
My initial PR that resulted in #1529 was not supporting format changes; If a filter was to convert a jpg to webp, then the versioned asset would still be served with the .jpg extension.
When applying the FormatExtensionResolver, this is now handled. Versioned JSON manifest assets get their extension swapped as well.
I've bumped the target version to 2.13.