Merge pull request #4161 from RedisInsight/fe/bugfix/CR-189-read-recommendations

CR-189 add configuration to trigger recommendation fetch on set interval

Author: pd-redis

redisinsight/ui/src/config/default.ts

@@ -33,6 +33,8 @@ export const defaultConfig = {
   database: {
     defaultConnectionTimeout: intEnv('RI_CONNECTIONS_TIMEOUT_DEFAULT', 30_000),
     defaultTimeoutToGetInfo: intEnv('RI_TIMEOUT_TO_GET_INFO', 5_000),
+    defaultTimeoutToGetRecommendations: intEnv('RI_TIMEOUT_TO_GET_RECOMMENDATIONS', 60_000),
+    shouldGetRecommendations: booleanEnv('RI_SHOULD_GET_RECOMMENDATIONS', false),
   },
   app: {
     env: process.env.NODE_ENV,

redisinsight/ui/src/pages/instance/InstancePage.tsx

@@ -26,9 +26,15 @@ import { resetConnectedInstance as resetRdiConnectedInstance } from 'uiSrc/slice
 import { loadPluginsAction } from 'uiSrc/slices/app/plugins'
 import { appConnectivityError } from 'uiSrc/slices/app/connectivity'
 import { appFeatureFlagsFeaturesSelector } from 'uiSrc/slices/app/features'
+import { getConfig } from 'uiSrc/config'
 import InstancePageRouter from './InstancePageRouter'
 import InstanceConnectionLost from './instanceConnectionLost'
 
+const riConfig = getConfig()
+
+const { shouldGetRecommendations,
+  defaultTimeoutToGetRecommendations } = riConfig.database
+
 export interface Props {
   routes: any[]
 }
@@ -58,6 +64,13 @@ const InstancePage = ({ routes = [] }: Props) => {
     dispatch(getDatabaseConfigInfoAction(connectionInstanceId))
     dispatch(fetchConnectedInstanceInfoAction(connectionInstanceId))
     dispatch(fetchRecommendationsAction(connectionInstanceId))
+    let intervalId: ReturnType<typeof setInterval>
+
+    if (shouldGetRecommendations) {
+      intervalId = setInterval(() => {
+        dispatch(fetchRecommendationsAction(connectionInstanceId))
+      }, defaultTimeoutToGetRecommendations)
+    }
 
     if (contextInstanceId && contextInstanceId !== connectionInstanceId) {
       // rerender children only if the same page from scratch to clear all component states
@@ -73,6 +86,10 @@ const InstancePage = ({ routes = [] }: Props) => {
 
     // clear rdi connection
     dispatch(resetRdiConnectedInstance())
+
+    return () => {
+      intervalId && clearInterval(intervalId)
+    }
   }, [connectionInstanceId])
 
   useEffect(() => {