Skip to content

Commit f50aa1b

Browse files
#RI-3198_update telemetry (#923)
* #RI-3198_update telemetry
1 parent 3963f8a commit f50aa1b

File tree

2 files changed

+15
-6
lines changed

2 files changed

+15
-6
lines changed

redisinsight/ui/src/pages/home/components/AddInstanceForm/InstanceForm/InstanceForm.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -169,6 +169,7 @@ const AddStandaloneForm = (props: Props) => {
169169
sentinelMasterUsername,
170170
isRediStack,
171171
servername,
172+
provider,
172173
},
173174
initialValues: initialValuesProp,
174175
width,
@@ -365,7 +366,9 @@ const AddStandaloneForm = (props: Props) => {
365366
sendEventTelemetry({
366367
event: TelemetryEvent.CONFIG_DATABASES_OPEN_DATABASE_BUTTON_CLICKED,
367368
eventData: {
368-
databaseId: id
369+
databaseId: id,
370+
provider,
371+
modules,
369372
}
370373
})
371374
dispatch(checkConnectToInstanceAction(id, connectToInstance))

redisinsight/ui/src/pages/home/components/DatabasesListComponent/DatabasesListWrapper.tsx

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -120,12 +120,17 @@ const DatabasesListWrapper = ({
120120
}
121121
history.push(Pages.browser(id))
122122
}
123-
const handleCheckConnectToInstance = (event: any, id = '') => {
123+
const handleCheckConnectToInstance = (
124+
event: React.MouseEvent | React.KeyboardEvent,
125+
{ id, provider, modules }: Instance
126+
) => {
124127
event.preventDefault()
125128
sendEventTelemetry({
126129
event: TelemetryEvent.CONFIG_DATABASES_OPEN_DATABASE,
127130
eventData: {
128-
databaseId: id
131+
databaseId: id,
132+
provider,
133+
modules,
129134
}
130135
})
131136
dispatch(checkConnectToInstanceAction(id, connectToInstance))
@@ -169,7 +174,8 @@ const DatabasesListWrapper = ({
169174
'data-test-subj': 'database-alias-column',
170175
sortable: ({ name }) => name?.toLowerCase(),
171176
width: '30%',
172-
render: function InstanceCell(name: string = '', { id, db }: Instance) {
177+
render: function InstanceCell(name: string = '', instance: Instance) {
178+
const { id, db } = instance
173179
const cellContent = replaceSpaces(name.substring(0, 200))
174180
return (
175181
<div
@@ -184,8 +190,8 @@ const DatabasesListWrapper = ({
184190
<EuiText
185191
className={styles.tooltipAnchorColumnName}
186192
data-testid={`instance-name-${id}`}
187-
onClick={(e: React.MouseEvent) => handleCheckConnectToInstance(e, id)}
188-
onKeyDown={(e: React.KeyboardEvent) => handleCheckConnectToInstance(e, id)}
193+
onClick={(e: React.MouseEvent) => handleCheckConnectToInstance(e, instance)}
194+
onKeyDown={(e: React.KeyboardEvent) => handleCheckConnectToInstance(e, instance)}
189195
>
190196
<EuiTextColor
191197
className={cx(styles.tooltipColumnNameText, { [styles.withDb]: db })}

0 commit comments

Comments
 (0)