Refresh configuration on page load

Signed-off-by: Siddhant Deshmukh <[email protected]>
opensearch-project · Nov 12, 2024 · fd67fb9 · fd67fb9
1 parent 9b3b60f
commit fd67fb9
Showing 1 changed file with 11 additions and 6 deletions.
diff --git a/public/pages/Configuration/Configuration.tsx b/public/pages/Configuration/Configuration.tsx
@@ -66,14 +66,19 @@ const Configuration = ({
   const [windowSize, setWindowSize] = useState(latencySettings.currWindowSize);
   const [time, setTime] = useState(latencySettings.currTimeUnit);
 
-  const metricSettingsMap = useMemo(
-    () => ({
+  const [metricSettingsMap, setMetricSettingsMap] = useState({
+    latency: latencySettings,
+    cpu: cpuSettings,
+    memory: memorySettings,
+  });
+
+  useEffect(() => {
+    setMetricSettingsMap({
       latency: latencySettings,
       cpu: cpuSettings,
       memory: memorySettings,
-    }),
-    [latencySettings, cpuSettings, memorySettings]
-  );
+    });
+  }, [latencySettings, cpuSettings, memorySettings]);
 
   const newOrReset = useCallback(() => {
     const currMetric = metricSettingsMap[metric];
@@ -85,7 +90,7 @@ const Configuration = ({
 
   useEffect(() => {
     newOrReset();
-  }, [newOrReset]);
+  }, [newOrReset, metricSettingsMap]);
 
   useEffect(() => {
     core.chrome.setBreadcrumbs([