Skip to content

feat(hyva): add Hyvä theme compatibility for address lookup#579

Merged
cblanc merged 3 commits into
mainfrom
hyva
Jun 3, 2026
Merged

feat(hyva): add Hyvä theme compatibility for address lookup#579
cblanc merged 3 commits into
mainfrom
hyva

Conversation

@mfilip

@mfilip mfilip commented Jun 2, 2026

Copy link
Copy Markdown
Member

Hyvä themes don't use RequireJS, so x-magento-init scripts aren't processed. This adds direct script loading for Hyvä pages via store-hyva.phtml template.

Also adds Docker config for Magento 2.4.8-p4 with PHP 8.4 and OpenSearch.

Hyvä themes don't use RequireJS, so x-magento-init scripts aren't processed.
This adds direct script loading for Hyvä pages via store-hyva.phtml template.

Also adds Docker config for Magento 2.4.8-p4 with PHP 8.4 and OpenSearch.
@mfilip mfilip requested a review from cblanc June 2, 2026 13:37
- Remove RequireJS dependency for universal theme compatibility (Luma, Hyvä, custom)
- Use external loader.js instead of inline scripts for CSP compliance
- Move block to before.body.end for better cross-theme support
- Config passed via data attributes for security
Comment thread view/frontend/web/js/loader.js Fixed
… in loader.js

Add security check to validate that binding URL is either relative or same-origin before loading script. Prevents potential XSS attacks by blocking external script sources.
Comment thread view/frontend/web/js/loader.js Dismissed
@cblanc cblanc merged commit 23c589a into main Jun 3, 2026
5 checks passed
@idealbot

idealbot commented Jun 3, 2026

Copy link
Copy Markdown
Member

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@idealbot

idealbot commented Jun 3, 2026

Copy link
Copy Markdown
Member

🎉 This PR is included in version 2.6.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

4 participants