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

Name ecommerce.items parameter names in purchase event GA4-compliant #7

Open
wants to merge 326 commits into
base: master
Choose a base branch
from

Conversation

wmdkfhartung
Copy link

Currently some of the items parameter names in the purchase event are not named GA4 compliant. (https://github.com/d3datadevelopment/GoogleAnalytics4/blob/master/Application/views/blocks/checkout_s5.tpl)

So they are called "id, name, variant", although they should be called "item_id, item_name, item_variant". (See https://developers.google.com/analytics/devguides/collection/ga4/set-up-ecommerce?hl=de#add-event)

In the view_item, add_to_cart and view_cart event they are named correctly.

MaxBuhe01 and others added 30 commits February 22, 2023 14:05
taken from pull-request; thank you C. Stäblein
# Conflicts:
#	CHANGELOG.md
#	metadata.php
MaxBuhe01 and others added 30 commits June 3, 2024 12:15
accidentally the existing settings were overwritten
accidentally the existing settings were overwritten
…_2.x_fix_onactivate

# Conflicts:
#	Setup/Actions.php
Activating the Module on CLI happens so early, there's no ModuleChain available yet, therefor no extension, therefor no additional methods
…ontroller\d3GtmStartController is not an instance of OxidEsales\Eshop\Application\Component\Widget\WidgetController"
- new block extension
- new template for the event-data for GA
- metadata adjustments for it
- new version ( 2.20.0 )
- add Payment-Model extension for own small method
- new block extension
- new template for the event-data for GA
- metadata adjustments for it
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.

5 participants