fix(SPM-2117): remove await inside react hook (#1089)

* fix(SPM-2117): remove await inside react hook * chore: bring back the useEffect dependencies * chore: remove unneseccary npm script
RedHatInsights · Jul 13, 2023 · cddb13e · cddb13e
1 parent 83f93b4
commit cddb13e
Showing 1 changed file with 9 additions and 7 deletions.
diff --git a/src/PresentationalComponents/StatusReports/SystemsStatusReport.js b/src/PresentationalComponents/StatusReports/SystemsStatusReport.js
@@ -1,4 +1,4 @@
-import React, { useMemo } from 'react';
+import React from 'react';
 import { useSelector } from 'react-redux';
 import { CheckCircleIcon, PackageIcon, ExclamationTriangleIcon } from '@patternfly/react-icons';
 import propTypes from 'prop-types';
@@ -46,27 +46,33 @@ const SystemsStatusReport = ({ apply, queryParams }) => {
 
     const { selectedTags, selectedGlobalTags, systemProfile } = useSelector(({ GlobalFilterStore }) => GlobalFilterStore);
 
-    const res = useMemo(async () =>{
+    const fetchResource = () => {
         setSubtotals({});
 
         let result;
 
         try {
-            result = await fetchSystems({
+            result = fetchSystems({
                 filter: {
                     os: queryParams?.filter?.os
                 },
                 selectedTags: [...selectedTags, ...selectedGlobalTags],
                 systemProfile,
                 limit: 1,
                 'filter[stale]': 'in:true,false'
+            }).then((result)=> {
+                setSubtotals(result.meta?.subtotals);
             });
         }
         catch {
             result = [];
         }
 
         return result;
+    };
+
+    React.useEffect(() => {
+        fetchResource();
     }, [
         queryParams?.filter?.os?.length, queryParams?.filter?.os !== undefined,
         selectedTags?.length, selectedTags !== undefined,
@@ -75,10 +81,6 @@ const SystemsStatusReport = ({ apply, queryParams }) => {
         queryParams?.subtotals !== undefined
     ]);
 
-    res.then((result)=> {
-        setSubtotals(result.meta?.subtotals);
-    });
-
     return (
         <Main style={{ paddingBottom: 0 }}>
             <Grid hasGutter span={12} >