From bcf883223b4bd9369b038ce060b34ee658a3e30e Mon Sep 17 00:00:00 2001 From: Br0wnHammer Date: Tue, 17 Jun 2025 13:13:21 +0530 Subject: [PATCH] Exporting Hooks from One File --- .../{checkHooksMonitor.js => checkHooks.js} | 53 +++++++++++++++++- client/src/Hooks/checkHooksTeam.js | 56 ------------------- .../Components/IncidentTable/index.jsx | 12 ++-- client/src/Pages/Uptime/Details/index.jsx | 2 +- 4 files changed, 59 insertions(+), 64 deletions(-) rename client/src/Hooks/{checkHooksMonitor.js => checkHooks.js} (52%) delete mode 100644 client/src/Hooks/checkHooksTeam.js diff --git a/client/src/Hooks/checkHooksMonitor.js b/client/src/Hooks/checkHooks.js similarity index 52% rename from client/src/Hooks/checkHooksMonitor.js rename to client/src/Hooks/checkHooks.js index e46adfa98..ed4cffe3f 100644 --- a/client/src/Hooks/checkHooksMonitor.js +++ b/client/src/Hooks/checkHooks.js @@ -2,6 +2,57 @@ import { useState, useEffect } from "react"; import { networkService } from "../main"; import { createToast } from "../Utils/toastUtils"; +const useFetchChecksTeam = ({ + teamId, + status, + sortOrder, + limit, + dateRange, + filter, + page, + rowsPerPage, +}) => { + const [checks, setChecks] = useState(undefined); + const [checksCount, setChecksCount] = useState(undefined); + const [isLoading, setIsLoading] = useState(false); + const [networkError, setNetworkError] = useState(false); + + useEffect(() => { + const fetchChecks = async () => { + if (!teamId) { + return; + } + + const config = { + status, + teamId, + sortOrder, + limit, + dateRange, + filter, + page, + rowsPerPage, + }; + + try { + setIsLoading(true); + const res = await networkService.getChecksByTeam(config); + setChecks(res.data.data.checks); + setChecksCount(res.data.data.checksCount); + } catch (error) { + setNetworkError(true); + createToast({ body: error.message }); + } finally { + setIsLoading(false); + } + }; + + fetchChecks(); + }, [teamId, status, sortOrder, limit, dateRange, filter, page, rowsPerPage]); + + return [checks, checksCount, isLoading, networkError]; +}; + const useFetchChecksByMonitor = ({ monitorId, type, @@ -55,4 +106,4 @@ const useFetchChecksByMonitor = ({ return [checks, checksCount, isLoading, networkError]; }; -export { useFetchChecksByMonitor }; +export { useFetchChecksByMonitor, useFetchChecksTeam }; diff --git a/client/src/Hooks/checkHooksTeam.js b/client/src/Hooks/checkHooksTeam.js deleted file mode 100644 index 05f8b67e5..000000000 --- a/client/src/Hooks/checkHooksTeam.js +++ /dev/null @@ -1,56 +0,0 @@ -import { useState, useEffect } from "react"; -import { networkService } from "../main"; -import { createToast } from "../Utils/toastUtils"; - -const useFetchChecksTeam = ({ - teamId, - status, - sortOrder, - limit, - dateRange, - filter, - page, - rowsPerPage, -}) => { - const [checks, setChecks] = useState(undefined); - const [checksCount, setChecksCount] = useState(undefined); - const [isLoading, setIsLoading] = useState(false); - const [networkError, setNetworkError] = useState(false); - - useEffect(() => { - const fetchChecks = async () => { - if (!teamId) { - return; - } - - const config = { - status, - teamId, - sortOrder, - limit, - dateRange, - filter, - page, - rowsPerPage, - }; - - try { - setIsLoading(true); - const res = await networkService.getChecksByTeam(config); - setChecks(res.data.data.checks); - setChecksCount(res.data.data.checksCount); - } catch (error) { - setNetworkError(true); - createToast({ body: error.message }); - } finally { - setIsLoading(false); - } - }; - - fetchChecks(); - }, [teamId, status, sortOrder, limit, dateRange, filter, page, rowsPerPage]); - - return [checks, checksCount, isLoading, networkError]; -}; - -export { useFetchChecksTeam }; diff --git a/client/src/Pages/Incidents/Components/IncidentTable/index.jsx b/client/src/Pages/Incidents/Components/IncidentTable/index.jsx index 97a006884..b36abad70 100644 --- a/client/src/Pages/Incidents/Components/IncidentTable/index.jsx +++ b/client/src/Pages/Incidents/Components/IncidentTable/index.jsx @@ -13,8 +13,8 @@ import { useSelector } from "react-redux"; import { useState } from "react"; import PropTypes from "prop-types"; import { useTranslation } from "react-i18next"; -import { useFetchChecksTeam } from "../../../../Hooks/checkHooksTeam"; -import { useFetchChecksByMonitor } from "../../../../Hooks/checkHooksMonitor"; +import { useFetchChecksTeam } from "../../../../Hooks/checkHooks"; +import { useFetchChecksByMonitor } from "../../../../Hooks/checkHooks"; const IncidentTable = ({ shouldRender, @@ -57,10 +57,10 @@ const IncidentTable = ({ rowsPerPage: rowsPerPage, }); - const checks = selectedMonitor === "0" ? checksTeam : checksMonitor; - const checksCount = selectedMonitor === "0" ? checksCountTeam : checksCountMonitor; - const isLoading = selectedMonitor === "0" ? isLoadingTeam : isLoadingMonitor; - const networkError = selectedMonitor === "0" ? networkErrorTeam : networkErrorMonitor; + const checks = checksTeam || checksMonitor; + const checksCount = checksCountTeam || checksCountMonitor; + const isLoading = isLoadingTeam || isLoadingMonitor; + const networkError = networkErrorTeam || networkErrorMonitor; const { t } = useTranslation(); diff --git a/client/src/Pages/Uptime/Details/index.jsx b/client/src/Pages/Uptime/Details/index.jsx index 1d6b9c392..5f36ee7e5 100644 --- a/client/src/Pages/Uptime/Details/index.jsx +++ b/client/src/Pages/Uptime/Details/index.jsx @@ -18,7 +18,7 @@ import { useTheme } from "@emotion/react"; import { useIsAdmin } from "../../../Hooks/useIsAdmin"; import { useFetchUptimeMonitorById } from "../../../Hooks/monitorHooks"; import useCertificateFetch from "./Hooks/useCertificateFetch"; -import { useFetchChecksByMonitor } from "../../../Hooks/checkHooksMonitor"; +import { useFetchChecksByMonitor } from "../../../Hooks/checkHooks"; import { useTranslation } from "react-i18next"; // Constants