Rely on refresh context to update stats independently of overview cards.

elastic · Oct 17, 2022 · 4975273 · 4975273
1 parent 4234417
commit 4975273
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/...ics/public/apps/synthetics/components/monitors_page/overview/overview/overview_status.tsx b/...ics/public/apps/synthetics/components/monitors_page/overview/overview/overview_status.tsx
@@ -9,8 +9,13 @@ import { EuiFlexGroup, EuiFlexItem, EuiPanel, EuiSpacer, EuiStat, EuiTitle } fro
 import { i18n } from '@kbn/i18n';
 import React, { useEffect } from 'react';
 import { useDispatch, useSelector } from 'react-redux';
-import { clearOverviewStatusErrorAction, selectOverviewStatus } from '../../../../state';
+import {
+  clearOverviewStatusErrorAction,
+  fetchOverviewStatusAction,
+  selectOverviewStatus,
+} from '../../../../state';
 import { kibanaService } from '../../../../../../utils/kibana_service';
+import { useSyntheticsRefreshContext } from '../../../../contexts';
 
 function title(t?: number) {
   return t ?? '-';
@@ -20,6 +25,12 @@ export function OverviewStatus() {
   const { status, statusError } = useSelector(selectOverviewStatus);
   const dispatch = useDispatch();
 
+  const { lastRefresh } = useSyntheticsRefreshContext();
+
+  useEffect(() => {
+    dispatch(fetchOverviewStatusAction.get());
+  }, [dispatch, lastRefresh]);
+
   useEffect(() => {
     if (statusError) {
       dispatch(clearOverviewStatusErrorAction());

diff --git a/...ins/synthetics/public/apps/synthetics/components/monitors_page/overview/overview_page.tsx b/...ins/synthetics/public/apps/synthetics/components/monitors_page/overview/overview_page.tsx
@@ -14,7 +14,6 @@ import { useEnablement } from '../../../hooks';
 import { useSyntheticsRefreshContext } from '../../../contexts/synthetics_refresh_context';
 import {
   fetchMonitorOverviewAction,
-  fetchOverviewStatusAction,
   selectOverviewState,
   selectServiceLocationsState,
 } from '../../../state';
@@ -53,7 +52,6 @@ export const OverviewPage: React.FC = () => {
 
   useEffect(() => {
     dispatch(fetchMonitorOverviewAction.get(pageState));
-    dispatch(fetchOverviewStatusAction.get());
   }, [dispatch, pageState]);
 
   const {