- Issue created by @steven jones
- Merge request !1Issue #3464016: Fix contrib using VERSION in library definitions. → (Open) created by steven jones
- Status changed to Needs review
4 months ago 1:46pm 26 July 2024 - 🇬🇧United Kingdom steven jones
This needs a review.
By default, it's going to try to add the module/theme version to the core version where
VERSION
is being used, but it can also add the deployment identifier and hash the whole lot, in case we want that. - 🇬🇧United Kingdom james.williams
I've reviewed the code and I think I'm happy, though I couldn't actually reproduce the issue locally, with this fixing it, in the first place. What conditions & steps would demonstrate it? I tried with CSS/JS aggregation on & off; with & without a deployment_identifier (and changing it); with & without caching enabled in devtools' network tab.