Skip to content

Commit 8264a44

Browse files
author
Artem
committed
Avoid using common id for non-tracking events such as APPLICATION_START etc.
1 parent b5d20b4 commit 8264a44

File tree

1 file changed

+1
-10
lines changed

1 file changed

+1
-10
lines changed

redisinsight/api/src/modules/core/services/analytics/analytics.service.ts

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ export class AnalyticsService {
6161
'agreements.analytics',
6262
false,
6363
);
64-
if (isAnalyticsGranted) {
64+
if (isAnalyticsGranted || nonTracking) {
6565
this.analytics.track({
6666
anonymousId: this.anonymousId,
6767
integrations: { Amplitude: { session_id: this.sessionId } },
@@ -70,15 +70,6 @@ export class AnalyticsService {
7070
...eventData,
7171
},
7272
});
73-
} else if (nonTracking) {
74-
this.analytics.track({
75-
anonymousId: NON_TRACKING_ANONYMOUS_ID,
76-
integrations: { Amplitude: { session_id: this.sessionId } },
77-
event,
78-
properties: {
79-
...eventData,
80-
},
81-
});
8273
}
8374
} catch (e) {
8475
// continue regardless of error

0 commit comments

Comments
 (0)