Update internal pixel gtag id#6554
Conversation
Greptile SummaryThis PR replaces the Google Analytics 4 measurement ID (
Confidence Score: 3/5The change substitutes a GA4 measurement ID with a Google Ads ID in a function whose name and likely implementation are tied to GA4 analytics, which could silently break analytics collection. The swap from a
Important Files Changed
Reviews (1): Last reviewed commit: "Update gtag id" | Re-trigger Greptile |
| """ | ||
| return [ | ||
| *get_google_analytics_trackers(tracking_id="G-4T7C8ZD9TR"), | ||
| *get_google_analytics_trackers(tracking_id="AW-18175470538"), |
There was a problem hiding this comment.
Google Ads ID passed to Analytics tracker
The new ID AW-18175470538 uses the AW- prefix, which identifies a Google Ads property (conversion linker / remarketing tag), whereas the previous ID G-4T7C8ZD9TR was a GA4 Analytics measurement ID (G- prefix). Passing a Google Ads ID to get_google_analytics_trackers may silently drop pageview/analytics events since that function is likely configured specifically to emit gtag('config', ...) calls suitable for GA4. GA4 analytics collection would be lost entirely. If the intent is to replace GA4 with a Google Ads tag, a dedicated function or a rename of the call site would make the intent explicit and avoid the mismatch.
Palash is gonna check this out |
No description provided.