Move BaseAppWrapper definition out of the useRootProfile hook

elastic · Oct 29, 2024 · 21ae097 · 21ae097
1 parent 0675b8d
commit 21ae097
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/plugins/discover/public/context_awareness/hooks/use_root_profile.tsx b/src/plugins/discover/public/context_awareness/hooks/use_root_profile.tsx
@@ -33,14 +33,12 @@ export const useRootProfile = () => {
         filter((id) => id !== undefined),
         tap(() => setRootProfileState({ rootProfileLoading: true })),
         switchMap((id) => profilesManager.resolveRootProfile({ solutionNavId: id })),
-        tap(({ getRenderAppWrapper }) => {
-          const BaseAppWrapper: Profile['getRenderAppWrapper'] = ({ children }) => <>{children}</>;
-
+        tap(({ getRenderAppWrapper }) =>
           setRootProfileState({
             rootProfileLoading: false,
             AppWrapper: getRenderAppWrapper?.(BaseAppWrapper) ?? BaseAppWrapper,
-          });
-        })
+          })
+        )
       )
       .subscribe();
 
@@ -51,3 +49,5 @@ export const useRootProfile = () => {
 
   return rootProfileState;
 };
+
+const BaseAppWrapper: Profile['getRenderAppWrapper'] = ({ children }) => <>{children}</>;