From 4fec00334ace441487594ef6ac2860c3f0212177 Mon Sep 17 00:00:00 2001
From: Alex Holliday <ajhollid@gmail.com>
Date: Fri, 7 Feb 2025 09:19:01 -0800
Subject: [PATCH] remove old loading state

---
 Client/src/Components/ActionsMenu/index.jsx       |  2 +-
 .../Monitors/Components/UptimeDataTable/index.jsx |  4 ----
 Client/src/Pages/Uptime/Monitors/index.jsx        | 15 +++++++--------
 3 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/Client/src/Components/ActionsMenu/index.jsx b/Client/src/Components/ActionsMenu/index.jsx
index a14e347fe..c39c130cb 100644
--- a/Client/src/Components/ActionsMenu/index.jsx
+++ b/Client/src/Components/ActionsMenu/index.jsx
@@ -18,7 +18,7 @@ const ActionsMenu = ({
 	isAdmin,
 	updateRowCallback,
 	pauseCallback,
-	setIsLoading,
+	setIsLoading = () => {},
 }) => {
 	const [anchorEl, setAnchorEl] = useState(null);
 	const [actions, setActions] = useState({});
diff --git a/Client/src/Pages/Uptime/Monitors/Components/UptimeDataTable/index.jsx b/Client/src/Pages/Uptime/Monitors/Components/UptimeDataTable/index.jsx
index 323365031..0b39e141b 100644
--- a/Client/src/Pages/Uptime/Monitors/Components/UptimeDataTable/index.jsx
+++ b/Client/src/Pages/Uptime/Monitors/Components/UptimeDataTable/index.jsx
@@ -49,7 +49,6 @@ import PropTypes from "prop-types";
 const UptimeDataTable = ({
 	isAdmin,
 	isSearching,
-	setIsLoading,
 	filteredMonitors,
 	sort,
 	setSort,
@@ -167,7 +166,6 @@ const UptimeDataTable = ({
 					monitor={row.monitor}
 					isAdmin={isAdmin}
 					updateRowCallback={triggerUpdate}
-					setIsLoading={setIsLoading}
 					pauseCallback={triggerUpdate}
 				/>
 			),
@@ -206,12 +204,10 @@ UptimeDataTable.propTypes = {
 	isSearching: PropTypes.bool,
 	setSort: PropTypes.func,
 	setSearch: PropTypes.func,
-	setIsLoading: PropTypes.func,
 	triggerUpdate: PropTypes.func,
 	debouncedSearch: PropTypes.string,
 	onSearchChange: PropTypes.func,
 	isAdmin: PropTypes.bool,
-	isLoading: PropTypes.bool,
 	monitors: PropTypes.array,
 	filteredMonitors: PropTypes.array,
 	monitorCount: PropTypes.number,
diff --git a/Client/src/Pages/Uptime/Monitors/index.jsx b/Client/src/Pages/Uptime/Monitors/index.jsx
index 0784b46a8..f8ce1332f 100644
--- a/Client/src/Pages/Uptime/Monitors/index.jsx
+++ b/Client/src/Pages/Uptime/Monitors/index.jsx
@@ -62,7 +62,6 @@ const UptimeMonitors = () => {
 	const [page, setPage] = useState(undefined);
 	const [sort, setSort] = useState(undefined);
 	const [isSearching, setIsSearching] = useState(false);
-	const [isLoading, setIsLoading] = useState(false);
 	const [monitorUpdateTrigger, setMonitorUpdateTrigger] = useState(false);
 
 	// Utils
@@ -107,8 +106,7 @@ const UptimeMonitors = () => {
 		order: sort?.order,
 		triggerUpdate: monitorUpdateTrigger,
 	});
-	const totalMonitors = monitorsSummary?.totalMonitors ?? 0;
-
+	const totalMonitors = monitorsSummary?.totalMonitors;
 	if (networkError) {
 		return (
 			<GenericFallback>
@@ -123,7 +121,10 @@ const UptimeMonitors = () => {
 			</GenericFallback>
 		);
 	}
-	if (!isLoading && !monitorsAreLoading && totalMonitors === 0) {
+	if (
+		!monitorsAreLoading &&
+		(totalMonitors === 0 || typeof totalMonitors === "undefined")
+	) {
 		return (
 			<Fallback
 				vowelStart={true}
@@ -134,7 +135,6 @@ const UptimeMonitors = () => {
 			/>
 		);
 	}
-
 	return (
 		<Stack
 			className="monitors"
@@ -143,7 +143,7 @@ const UptimeMonitors = () => {
 			<Breadcrumbs list={BREADCRUMBS} />
 			<CreateMonitorHeader
 				isAdmin={isAdmin}
-				shouldRender={!isLoading}
+				shouldRender={!monitorsAreLoading}
 				path="/uptime/create"
 			/>
 			<Greeting type="uptime" />
@@ -166,8 +166,7 @@ const UptimeMonitors = () => {
 			</Stack>
 			<UptimeDataTable
 				isAdmin={isAdmin}
-				isLoading={isLoading}
-				setIsLoading={setIsLoading}
+				isLoading={monitorsAreLoading}
 				filteredMonitors={filteredMonitors}
 				monitors={monitors}
 				monitorCount={totalMonitors}