Skip to content

Commit 6e07f3f

Browse files
authored
Merge pull request #1654 from RedisInsight/fe/bugfix/RI-4122
#RI-4122 - fix redisearch module property loaded for telemetry
2 parents d6d131a + 3ec7047 commit 6e07f3f

File tree

1 file changed

+14
-3
lines changed

1 file changed

+14
-3
lines changed

redisinsight/ui/src/telemetry/telemetryUtils.ts

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
import isGlob from 'is-glob'
66
import { cloneDeep } from 'lodash'
77
import * as jsonpath from 'jsonpath'
8-
import { Nullable } from 'uiSrc/utils'
8+
import { isRedisearchAvailable, Nullable } from 'uiSrc/utils'
99
import { localStorageService } from 'uiSrc/services'
1010
import { ApiEndpoints, BrowserStorageItem, KeyTypes, StreamViews } from 'uiSrc/constants'
1111
import { KeyViewType } from 'uiSrc/slices/interfaces/keys'
@@ -228,9 +228,20 @@ const getRedisModulesSummary = (modules: AdditionalRedisModule[] = []): IRedisMo
228228
version: module.version,
229229
semanticVersion: module.semanticVersion,
230230
}
231-
} else {
232-
summary.customModules.push(module)
231+
return
233232
}
233+
234+
if (isRedisearchAvailable([module])) {
235+
const redisearchName = getEnumKeyBValue(RedisModules, RedisModules.RediSearch)
236+
summary[redisearchName] = {
237+
loaded: true,
238+
version: module.version,
239+
semanticVersion: module.semanticVersion,
240+
}
241+
return
242+
}
243+
244+
summary.customModules.push(module)
234245
}))
235246
} catch (e) {
236247
// continue regardless of error

0 commit comments

Comments
 (0)