Merge pull request #2867 from Hyperkid123/disable-ocp-ins-va

Hide OpenShift VA behind a feature flag.
RedHatInsights · Jun 18, 2024 · 45d02dc · 45d02dc
2 parents 24026e7 + 6891edb
commit 45d02dc
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 9 deletions.
diff --git a/src/components/RootApp/ScalprumRoot.test.js b/src/components/RootApp/ScalprumRoot.test.js
@@ -38,6 +38,7 @@ jest.mock('@unleash/proxy-client-react', () => {
     ...unleash,
     useFlag: () => false,
     useFlagsStatus: () => ({ flagsReady: true }),
+    useFlags: () => [],
   };
 });
 

diff --git a/src/components/Routes/VirtualAssistant.tsx b/src/components/Routes/VirtualAssistant.tsx
@@ -1,20 +1,22 @@
 import React, { Fragment } from 'react';
 import { Route, Routes } from 'react-router-dom';
 import { ScalprumComponent } from '@scalprum/react-core';
+import { useFlags } from '@unleash/proxy-client-react';
 
 import './virtual-assistant.scss';
 
-const viableRoutes = [
-  '/',
-  '/insights/*',
-  '/settings/*',
-  '/subscriptions/overview/*',
-  '/subscriptions/inventory/*',
-  '/subscriptions/usage/*',
-  '/openshift/insights/*',
-];
+const flaggedRoutes: { [flagName: string]: string } = { 'platform.va.openshift.insights': '/openshift/insights/*' };
 
 const VirtualAssistant = () => {
+  const viableRoutes = ['/', '/insights/*', '/settings/*', '/subscriptions/overview/*', '/subscriptions/inventory/*', '/subscriptions/usage/*'];
+
+  const allFlags = useFlags();
+  allFlags.forEach((flag) => {
+    if (flaggedRoutes[flag.name] && flag.enabled) {
+      viableRoutes.push(flaggedRoutes[flag.name]);
+    }
+  });
+
   return (
     <Routes>
       {viableRoutes.map((route) => (