Skip to content

Commit 07c5cb4

Browse files
authored
Merge pull request #1902 from SUI-Components/fix/asyncGA4
fix(packages/sui-segment-wrapper): Remove adobeCloudVisitorId from GA4
2 parents 08c887c + 2f65d7f commit 07c5cb4

File tree

3 files changed

+15
-12
lines changed

3 files changed

+15
-12
lines changed
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
export const EVENTS = {
2+
GA4_INIT_EVENT_SENT: 'ga4InitEventSent'
3+
}

packages/sui-segment-wrapper/src/index.js

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,12 @@ import {defaultContextProperties} from './middlewares/source/defaultContextPrope
55
import {pageReferrer} from './middlewares/source/pageReferrer.js'
66
import {userScreenInfo} from './middlewares/source/userScreenInfo.js'
77
import {userTraits} from './middlewares/source/userTraits.js'
8+
import {getCampaignDetails, loadGoogleAnalytics} from './repositories/googleRepository.js'
89
import {checkAnonymousId} from './utils/checkAnonymousId.js'
910
import {getConfig, isClient} from './config.js'
1011
import analytics from './segmentWrapper.js'
1112
import initTcfTracking from './tcf.js'
1213
import {getUserDataAndNotify} from './universalId.js'
13-
import {loadGoogleAnalytics, getCampaignDetails} from './repositories/googleRepository.js'
14-
import {getAdobeMCVisitorID} from './repositories/adobeRepository.js'
1514

1615
// Initialize TCF Tracking with Segment
1716
initTcfTracking()
@@ -47,15 +46,11 @@ if (isClient && window.analytics) {
4746
}
4847

4948
window.gtag('js', new Date())
50-
51-
getAdobeMCVisitorID().then(marketingCloudVisitorId => {
52-
window.gtag('config', googleAnalyticsMeasurementId, {
53-
cookie_prefix: 'segment',
54-
send_page_view: false,
55-
...googleAnalyticsConfig,
56-
...getCampaignDetails(),
57-
mcvid: marketingCloudVisitorId
58-
})
49+
window.gtag('config', googleAnalyticsMeasurementId, {
50+
cookie_prefix: 'segment',
51+
send_page_view: false,
52+
...googleAnalyticsConfig,
53+
...getCampaignDetails()
5954
})
6055
loadGoogleAnalytics().catch(error => {
6156
console.error(error)
@@ -69,3 +64,4 @@ if (isClient && window.analytics) {
6964
export default analytics
7065
export {getAdobeVisitorData, getAdobeMCVisitorID} from './repositories/adobeRepository.js'
7166
export {getUniversalId} from './universalId.js'
67+
export {EVENTS} from './events.js'

packages/sui-segment-wrapper/src/repositories/googleRepository.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
1-
import {utils} from '../middlewares/source/pageReferrer.js'
1+
import {dispatchEvent} from '@s-ui/js/lib/events'
2+
23
import {getConfig} from '../config.js'
4+
import {EVENTS} from '../events.js'
5+
import {utils} from '../middlewares/source/pageReferrer.js'
36

47
const FIELDS = {
58
clientId: 'client_id',
@@ -60,6 +63,7 @@ const triggerGoogleAnalyticsInitEvent = sessionId => {
6063

6164
// And then save a new GA session hit in local storage.
6265
localStorage.setItem(eventKey, 'true')
66+
dispatchEvent({eventName: EVENTS.GA4_INIT_EVENT_SENT, detail: {eventName, sessionId}})
6367
}
6468

6569
// Clean old GA sessions hits from the storage.

0 commit comments

Comments
 (0)