Skip to content

Commit 2071f9f

Browse files
authored
Merge pull request #61 from RedisInsight/bugfix/RI-2063
#RI-2063 - add validation for command groups
2 parents aa5d267 + b121f46 commit 2071f9f

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

redisinsight/api/src/constants/commands/redistimeseries.json

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,6 +122,8 @@
122122
"multiple": true,
123123
"optional": true
124124
}
125-
]
125+
],
126+
"since": "1.0.0",
127+
"group": "timeseries"
126128
}
127129
}

redisinsight/ui/src/slices/app/redis-commands.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { createSlice } from '@reduxjs/toolkit'
2-
import { uniqBy } from 'lodash'
2+
import { isString, uniqBy } from 'lodash'
33
import { apiService } from 'uiSrc/services'
44
import { ApiEndpoints, ICommand, ICommands } from 'uiSrc/constants'
55
import { getApiErrorMessage, isStatusSuccessful } from 'uiSrc/utils'
@@ -30,6 +30,7 @@ const appRedisCommandsSlice = createSlice({
3030
state.commandsArray = Object.keys(state.spec).sort()
3131
state.commandGroups = uniqBy(Object.values(payload), 'group')
3232
.map((item: ICommand) => item.group)
33+
.filter((group: string) => isString(group))
3334
},
3435
getRedisCommandsFailure: (state, { payload }) => {
3536
state.loading = false

0 commit comments

Comments
 (0)