Skip to content

Commit b50d436

Browse files
committed
fix: we were not setting ga user id causing usage inconcsistencies
1 parent d1c5090 commit b50d436

2 files changed

Lines changed: 17 additions & 2 deletions

File tree

src/desktop-metrics.html

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,9 @@
1111
}
1212

1313
const SAVED_GOOGLE_ANALYTICS_DESKTOP_KEY = "SAVED_GOOGLE_ANALYTICS_DESKTOP_KEY";
14+
const SAVED_CUSTOM_USER_ID_KEY = "SAVED_CUSTOM_USER_ID";
1415
let gaReady = false, loadedGoogleAnalyticsID;
16+
let savedCustomUserID = localStorage.getItem(SAVED_CUSTOM_USER_ID_KEY);
1517

1618
const url = window.location.href;
1719
const urlParams = new URLSearchParams(window.location.search);
@@ -90,7 +92,11 @@
9092
document.head.appendChild(gaScript);
9193

9294
gtag('js', new Date());
93-
gtag('config', analyticsID);
95+
const configParams = {};
96+
if (savedCustomUserID) {
97+
configParams.client_id = savedCustomUserID;
98+
}
99+
gtag('config', analyticsID, configParams);
94100
gaReady = true;
95101
}
96102

@@ -109,6 +115,16 @@
109115

110116
async function processRequest(event) {
111117
const payload = event.payload;
118+
if (payload.customUserID && payload.customUserID !== savedCustomUserID) {
119+
savedCustomUserID = payload.customUserID;
120+
localStorage.setItem(SAVED_CUSTOM_USER_ID_KEY, payload.customUserID);
121+
if (gaReady) {
122+
gtag('config', loadedGoogleAnalyticsID, {
123+
client_id: savedCustomUserID,
124+
send_page_view: false
125+
});
126+
}
127+
}
112128
if(payload.analyticsID && loadedGoogleAnalyticsID !== payload.analyticsID) {
113129
localStorage.setItem(SAVED_GOOGLE_ANALYTICS_DESKTOP_KEY, payload.analyticsID);
114130
installGoogleAnalytics();

src/utils/Metrics.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -254,7 +254,6 @@ define(function (require, exports, module) {
254254
script.onload = function(){
255255
gtag('js', new Date());
256256

257-
// TODO use googleAnalyticsIDDesktop for desktop analytics
258257
gtag('config', brackets.config.googleAnalyticsID, {
259258
'page_title': 'Phoenix editor',
260259
'page_path': '/index.html',

0 commit comments

Comments
 (0)