Skip to content

Commit 91f428e

Browse files
authored
Merge pull request #3780 from RedisInsight/be/bugfix/RI-6073-fix-analytics-field-name
rename instanceId to databaseId
2 parents 43baecf + ee8ce87 commit 91f428e

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

redisinsight/api/src/modules/database/database.analytics.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -307,7 +307,7 @@ describe('DatabaseAnalytics', () => {
307307
expect(sendEventSpy).toHaveBeenCalledWith(
308308
TelemetryEvents.DatabaseConnectedClientList,
309309
{
310-
instanceId: mockDatabase.id,
310+
databaseId: mockDatabase.id,
311311
version: mockDatabase.version,
312312
resp: '2',
313313
},

redisinsight/api/src/modules/database/database.analytics.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -116,12 +116,12 @@ export class DatabaseAnalytics extends TelemetryBaseService {
116116
);
117117
}
118118

119-
sendDatabaseConnectedClientListEvent(instanceId: string, additionalData: object = {}): void {
119+
sendDatabaseConnectedClientListEvent(databaseId: string, additionalData: object = {}): void {
120120
try {
121121
this.sendEvent(
122122
TelemetryEvents.DatabaseConnectedClientList,
123123
{
124-
instanceId,
124+
databaseId,
125125
...additionalData,
126126
},
127127
);

0 commit comments

Comments
 (0)