Merge pull request #484 from PeterYurkovich/new-update

OU-943: remove broken useEffect

Author: openshift-merge-bot[bot]

web/src/components/MetricsPage.tsx

@@ -327,17 +327,6 @@ export const ToggleGraph: React.FC = () => {
 
   const dispatch = useDispatch();
   const toggle = React.useCallback(() => dispatch(toggleGraphs()), [dispatch]);
-  // Use an empty useEffect to get access to the cleanup function so that if graphs are
-  // currently hidden then we toggle one more time as we unmount
-  React.useEffect(
-    () => () => {
-      if (hideGraphs) {
-        toggle();
-      }
-    },
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-    [hideGraphs],
-  );
 
   // Use an empty useEffect to get access to the cleanup function so that if graphs are
   // currently hidden then we show the graphs as we unmount