Skip to content

Commit 1296b2e

Browse files
#RI-3313-add check for undefined keyname (#939)
1 parent 1e51006 commit 1296b2e

File tree

1 file changed

+15
-12
lines changed

1 file changed

+15
-12
lines changed

redisinsight/ui/src/pages/browser/components/key-details/KeyDetailsWrapper.tsx

Lines changed: 15 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import React, { useEffect } from 'react'
2+
import { isUndefined } from 'lodash'
23
import { useDispatch, useSelector } from 'react-redux'
34
import { useParams } from 'react-router-dom'
45

@@ -62,18 +63,20 @@ const KeyDetailsWrapper = (props: Props) => {
6263
}, [keyProp])
6364

6465
useEffect(() => {
65-
sendEventTelemetry({
66-
event: getBasedOnViewTypeEvent(
67-
viewType,
68-
TelemetryEvent.BROWSER_KEY_VALUE_VIEWED,
69-
TelemetryEvent.TREE_VIEW_KEY_VALUE_VIEWED
70-
),
71-
eventData: {
72-
keyType,
73-
databaseId: instanceId,
74-
length: keyLength,
75-
}
76-
})
66+
if (!isUndefined(keyName)) {
67+
sendEventTelemetry({
68+
event: getBasedOnViewTypeEvent(
69+
viewType,
70+
TelemetryEvent.BROWSER_KEY_VALUE_VIEWED,
71+
TelemetryEvent.TREE_VIEW_KEY_VALUE_VIEWED
72+
),
73+
eventData: {
74+
keyType,
75+
databaseId: instanceId,
76+
length: keyLength,
77+
}
78+
})
79+
}
7780
}, [keyName])
7881

7982
const handleDeleteKey = (key: string, type: string) => {

0 commit comments

Comments
 (0)