avoid duplicating context

elastic · Oct 3, 2024 · 7c13eba · 7c13eba
1 parent 16ac871
commit 7c13eba
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/.../observability_solution/investigate_app/public/pages/details/hooks/use_screen_context.tsx b/.../observability_solution/investigate_app/public/pages/details/hooks/use_screen_context.tsx
@@ -25,10 +25,10 @@ export function useScreenContext() {
   const { investigation } = useInvestigation();
   const alertOriginInvestigation = alertOriginSchema.safeParse(investigation?.origin);
   const alertId = alertOriginInvestigation.success ? alertOriginInvestigation.data.id : undefined;
-  const { data: alertDetails } = useFetchAlert({ id: alertId });
+  const { data: alertDetails, isLoading: isAlertDetailsLoading } = useFetchAlert({ id: alertId });
 
   useEffect(() => {
-    if (!investigation) {
+    if (!investigation || isAlertDetailsLoading) {
       return;
     }
 
@@ -53,7 +53,7 @@ export function useScreenContext() {
         },
       ],
     });
-  }, [observabilityAIAssistant, investigation, alertDetails]);
+  }, [observabilityAIAssistant, investigation, alertDetails, isAlertDetailsLoading]);
 }
 
 function getAlertDetailScreenContext(alertDetail: EcsFieldsResponse) {