fixed warnings

Author: talhainvenxion

src/api/runs/getGraphRunByIdApi.ts

@@ -2,7 +2,6 @@ import { fetchApiWithAuthRequest } from '../fetchApi';
 import { endpoints } from '../endpoints';
 import { httpMethods } from '../constants';
 import { apiUrl } from '../apiUrl';
-import mockApi from '../mockApiData';
 
 const getGraphRunByIdApi = ({
   authenticationToken,

src/ui/layouts/pipelines/RunDetail/useService.ts

@@ -6,12 +6,7 @@ import {
   runPagesSelectors,
   runSelectors,
 } from '../../../../redux/selectors';
-import {
-  useDispatch,
-  useParams,
-  useRequestOnMount,
-  useSelector,
-} from '../../../hooks';
+import { useDispatch, useParams, useSelector } from '../../../hooks';
 
 interface ServiceInterface {
   runId: TId;
@@ -44,6 +39,7 @@ export const useService = (): ServiceInterface => {
       );
       setIsMounted(true);
     }
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [isMounted, setIsMounted]);
   // useRequestOnMount(() =>
   //   runsActions.graphForRun({
@@ -68,11 +64,10 @@ export const useService = (): ServiceInterface => {
   //     pipelineId,
   //   }),
   // );
-
-  const fetching = useSelector(runPagesSelectors.fetching);
   const setFetching = (fetching: boolean) => {
     dispatch(runPagesActions.setFetching({ fetching }));
   };
+  const fetching = useSelector(runPagesSelectors.fetching);
 
   const billing = useSelector(billingSelectors.billingForRunId(id));
 

src/ui/layouts/runs/RunDetail/useService.ts

@@ -7,12 +7,7 @@ import {
   runPagesSelectors,
   runSelectors,
 } from '../../../../redux/selectors';
-import {
-  useDispatch,
-  useParams,
-  useRequestOnMount,
-  useSelector,
-} from '../../../hooks';
+import { useDispatch, useParams, useSelector } from '../../../hooks';
 
 interface ServiceInterface {
   runId: TId;
@@ -50,6 +45,7 @@ export const useService = (): ServiceInterface => {
       );
       setIsMounted(true);
     }
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [isMounted, setIsMounted]);
 
   // useRequestOnMount(() =>

src/ui/layouts/stackComponents/RunDetail/useService.ts

@@ -7,12 +7,7 @@ import {
   runPagesSelectors,
   runSelectors,
 } from '../../../../redux/selectors';
-import {
-  useDispatch,
-  useParams,
-  useRequestOnMount,
-  useSelector,
-} from '../../../hooks';
+import { useDispatch, useParams, useSelector } from '../../../hooks';
 
 interface ServiceInterface {
   runId: TId;
@@ -51,6 +46,7 @@ export const useService = (): ServiceInterface => {
       );
       setIsMounted(true);
     }
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [isMounted, setIsMounted]);
 
   // useRequestOnMount(() =>

src/ui/layouts/stacks/RunDetail/useService.ts

@@ -9,7 +9,7 @@ import {
 import {
   useDispatch,
   useParams,
-  useRequestOnMount,
+  // useRequestOnMount,
   useSelector,
 } from '../../../hooks';
 
@@ -38,19 +38,20 @@ export const useService = (): ServiceInterface => {
   //     onFailure: () => setFetching(false),
   //   }),
   // );
-  // useEffect(() => {
-  //   if (!isMounted) {
-  //     setFetching(true);
-  //     dispatch(
-  //       runsActions.graphForRun({
-  //         runId: id,
-  //         onSuccess: () => setFetching(false),
-  //         onFailure: () => setFetching(false),
-  //       }),
-  //     );
-  //     setIsMounted(true);
-  //   }
-  // }, [isMounted, setIsMounted]);
+  useEffect(() => {
+    if (!isMounted) {
+      setFetching(true);
+      dispatch(
+        runsActions.graphForRun({
+          runId: id,
+          onSuccess: () => setFetching(false),
+          onFailure: () => setFetching(false),
+        }),
+      );
+      setIsMounted(true);
+    }
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [isMounted, setIsMounted]);
 
   // useEffect(() => {
   //   setFetching(true);