fix: Fix non-working RBACProvider on empty appName

This fixes the RBACProvider component: now, with the empty appName property, it fetches permissions and update the state properly.
RedHatInsights · Aug 28, 2023 · ec4f350 · ec4f350
1 parent a5e01d5
commit ec4f350
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/packages/components/src/RBACProvider/RBACProvider.tsx b/packages/components/src/RBACProvider/RBACProvider.tsx
@@ -19,8 +19,8 @@ const hasAccessWithUserPermissions = (userPermissions: (Access | string)[], chec
 };
 
 export interface RBACProviderProps {
-  appName: string;
-  checkResourceDefinitions: boolean;
+  appName?: string;
+  checkResourceDefinitions?: boolean;
 }
 
 export const RBACProvider: React.FunctionComponent<RBACProviderProps> = ({ appName, checkResourceDefinitions = false, children }) => {
@@ -38,9 +38,7 @@ export const RBACProvider: React.FunctionComponent<RBACProviderProps> = ({ appNa
   };
 
   useEffect(() => {
-    if (appName) {
-      fetchPermissions();
-    }
+    fetchPermissions();
   }, [appName]);
 
   return (