Closed zixiang2018 closed 1 year ago
This pull request introduces 3 alerts when merging 1b8c96751645521ee6b8afef5e4c6093db0cc8d1 into 8348ca736498acb34f11a1a1a80f27277bfcc98a - view on LGTM.com
new alerts:
Before we can merge this to production, we'll need to confirm whether the same report can be generated in Data Studio with the data source now coming from GA4. Do you need help with that?
Context
To migrate Google Universal Analytics to Google Analytics 4 as "
Universal Analytics (UA) will be deprecated on July 1, 2023, which means it will stop processing data. Analytics 360 properties will stop working on October 1, 2023
"What this PR does
next-ga
withreact-ga4
- replaced using ofwindow.ga
withreact-ga4
functionslabel
key fromEvent
- GA4 does not support long string and label contains values which are already logged in our GA eventsGA_ID
with env variableGTAG_ID
GTAG_ID
into dev and prod env variabledocumentStore
with quotes before sending event to prevent GA4 from converting hexadecimal to decimal (related to this)On analytics.google.com