fix(graphCardContext): sw-690 duplicate api calls (#1012)

RedHatInsights · Nov 30, 2022 · 2891b10 · 2891b10
1 parent 339f43f
commit 2891b10
Showing 1 changed file with 15 additions and 10 deletions.
diff --git a/src/components/graphCard/graphCardContext.js b/src/components/graphCard/graphCardContext.js
@@ -100,19 +100,24 @@ const useGetMetrics = ({
   const { settings = {} } = useAliasGraphCardContext();
   const { metrics = [] } = settings;
 
-  const response = useAliasMetricsSelector();
+  const { pending, ...response } = useAliasMetricsSelector();
 
   useShallowCompareEffect(() => {
-    const updatedMetrics = metrics.map(({ metric: metricId, isCapacity, query: metricQuery }) => ({
-      id: productId,
-      metric: metricId,
-      isCapacity,
-      query: metricQuery
-    }));
-    getGraphMetrics(updatedMetrics, query)(dispatch);
-  }, [dispatch, getGraphMetrics, metrics, productId, query]);
+    if (!pending) {
+      const updatedMetrics = metrics.map(({ metric: metricId, isCapacity, query: metricQuery }) => ({
+        id: productId,
+        metric: metricId,
+        isCapacity,
+        query: metricQuery
+      }));
+      getGraphMetrics(updatedMetrics, query)(dispatch);
+    }
+  }, [metrics, productId, query]);
 
-  return response;
+  return {
+    ...response,
+    pending
+  };
 };
 
 const context = {