Skip to content

Commit 917e096

Browse files
authored
Merge pull request #1842 from RedisInsight/fe/feature/RI-4258_update_recommendations_telemetry
#RI-4258 - add list field in recommendations telemetry
2 parents 361f74c + f223d0c commit 917e096

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

redisinsight/ui/src/pages/databaseAnalysis/components/data-nav-tabs/DatabaseAnalysisTabs.spec.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,6 +125,7 @@ describe('DatabaseAnalysisTabs', () => {
125125
eventData: {
126126
databaseId: INSTANCE_ID_MOCK,
127127
recommendationsCount: 0,
128+
list: [],
128129
}
129130
})
130131
sendEventTelemetry.mockRestore()
@@ -135,7 +136,7 @@ describe('DatabaseAnalysisTabs', () => {
135136
sendEventTelemetry.mockImplementation(() => sendEventTelemetryMock)
136137

137138
const mockData = {
138-
recommendations: [{ name: 'luaScript' }, { name: 'luaScript' }]
139+
recommendations: [{ name: 'luaScript' }, { name: 'bigHashes' }]
139140
}
140141
render(<DatabaseAnalysisTabs {...instance(mockedProps)} reports={mockReports} data={mockData} />)
141142

@@ -146,6 +147,7 @@ describe('DatabaseAnalysisTabs', () => {
146147
eventData: {
147148
databaseId: INSTANCE_ID_MOCK,
148149
recommendationsCount: 2,
150+
list: ['luaScript', 'bigHashes'],
149151
}
150152
})
151153
sendEventTelemetry.mockRestore()

redisinsight/ui/src/pages/databaseAnalysis/components/data-nav-tabs/DatabaseAnalysisTabs.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ const DatabaseAnalysisTabs = (props: Props) => {
4747
eventData: {
4848
databaseId: instanceId,
4949
recommendationsCount: data?.recommendations?.length,
50+
list: data?.recommendations?.map(({ name }) => name),
5051
}
5152
})
5253
}

0 commit comments

Comments
 (0)