-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Fix wrong reference in google analytics module layout xml #18290
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 wrong reference in google analytics module layout xml #18290
Conversation
Hi @sambolek. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Although referenceContainer
works perfectly with blocks (at least on my instance), I do approve the fix to avoid inconsistencies with overriding it.
Hi @slavvka, thank you for the review. |
Hi @sambolek. Thank you for your contribution. |
Hi @sambolek , we need the fix to be delivered also to 2.2-develop branch so please don't forget to port this PR to 2.2-develop; you can do it easily using the Porting Tool, https://devdocs.magento.com/guides/v2.2/contributor-guide/contributing.html#porting-tool |
Hi @slavvka , |
@sambolek Thank you! |
Description
Simple change of reference (referenceContainer to referenceBlock) for Google Analytics module.
Fixed Issues (if relevant)
Manual testing scenarios
Contribution checklist