Open ramboz opened 3 months ago
Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed. In case there are problems, just click a checkbox below to rerun the respective action.
Page | Scores | Audits | |
---|---|---|---|
/ |
@fe-lix- @ekremney @bohnertchris Since I did quite some heavy lib-analytics.js
refactoring for this, if you know the various analytics events they were tracking before, could you perform a quick sanity check to make sure we didn't lose anything along the way?
@ramboz it has been almost 1 year since the custom analytics events were implemented for this project. I am afraid I cannot recall all the details. The most important events were tracked by GA4 via trackGTMEvent
This PR adds integration with Target, Launch & ACDL via the martech library. It also moves the Analytics instrumentation to the martech library, and creates a more generic loader to combine the GTM and WebSDK logic and isolate it from the main
scripts.js
file.At a high level:
lib-analytics.js
is renamed tolib-martech-loaded.js
*.petwatch.com
=> prod*.live
/*.page
urls => stagetrackAnalyticsEvent
method is exported to specifically track events in Analytics instead of GTMplugins/martech
head.html
gets new preload/preconnect links for resources immediately imported, so we don't have to wait forscripts.js
to be parsed.Test URLs: