From 720d1f74340b087c3b6233779a7da8de4083f369 Mon Sep 17 00:00:00 2001
From: Alex Holliday <ajhollid@gmail.com>
Date: Tue, 4 Feb 2025 16:01:46 -0800
Subject: [PATCH] don't render control buttons for public status page

---
 .../Components/ControlsHeader/index.jsx       | 97 ++++++++++++-------
 .../Status/Components/StatusBar/index.jsx     |  2 +-
 2 files changed, 64 insertions(+), 35 deletions(-)

diff --git a/Client/src/Pages/StatusPage/Status/Components/ControlsHeader/index.jsx b/Client/src/Pages/StatusPage/Status/Components/ControlsHeader/index.jsx
index f142392ad..c39d1d396 100644
--- a/Client/src/Pages/StatusPage/Status/Components/ControlsHeader/index.jsx
+++ b/Client/src/Pages/StatusPage/Status/Components/ControlsHeader/index.jsx
@@ -6,10 +6,62 @@ import SettingsIcon from "../../../../../assets/icons/settings-bold.svg?react";
 //Utils
 import { useTheme } from "@mui/material/styles";
 import { useNavigate } from "react-router-dom";
+import { useLocation } from "react-router-dom";
+import PropTypes from "prop-types";
+
+const Controls = ({ deleteStatusPage, isDeleting }) => {
+	const theme = useTheme();
+	const location = useLocation();
+	const currentPath = location.pathname;
+	const navigate = useNavigate();
+	if (currentPath === "/status/public") {
+		return null;
+	}
+
+	return (
+		<Stack
+			direction="row"
+			gap={theme.spacing(2)}
+		>
+			<Box>
+				<Button
+					variant="contained"
+					color="error"
+					onClick={deleteStatusPage}
+					loading={isDeleting}
+				>
+					Delete
+				</Button>
+			</Box>
+			<Box>
+				<Button
+					variant="contained"
+					color="secondary"
+					onClick={() => navigate(`/status/create`)}
+					sx={{
+						px: theme.spacing(5),
+						"& svg": {
+							mr: theme.spacing(3),
+							"& path": {
+								stroke: theme.palette.secondary.contrastText,
+							},
+						},
+					}}
+				>
+					<SettingsIcon /> Configure
+				</Button>
+			</Box>
+		</Stack>
+	);
+};
+
+Controls.propTypes = {
+	deleteStatusPage: PropTypes.func,
+	isDeleting: PropTypes.bool,
+};
 
 const ControlsHeader = ({ statusPage, deleteStatusPage, isDeleting }) => {
 	const theme = useTheme();
-	const navigate = useNavigate();
 
 	return (
 		<Stack
@@ -33,41 +85,18 @@ const ControlsHeader = ({ statusPage, deleteStatusPage, isDeleting }) => {
 				/>
 				<Typography variant="h2">{statusPage?.companyName}</Typography>
 			</Stack>
-			<Stack
-				direction="row"
-				gap={theme.spacing(2)}
-			>
-				<Box>
-					<Button
-						variant="contained"
-						color="error"
-						onClick={deleteStatusPage}
-						loading={isDeleting}
-					>
-						Delete
-					</Button>
-				</Box>
-				<Box>
-					<Button
-						variant="contained"
-						color="secondary"
-						onClick={() => navigate(`/status/create`)}
-						sx={{
-							px: theme.spacing(5),
-							"& svg": {
-								mr: theme.spacing(3),
-								"& path": {
-									stroke: theme.palette.secondary.contrastText,
-								},
-							},
-						}}
-					>
-						<SettingsIcon /> Configure
-					</Button>
-				</Box>
-			</Stack>
+			<Controls
+				deleteStatusPage={deleteStatusPage}
+				isDeleting={isDeleting}
+			/>
 		</Stack>
 	);
 };
 
+ControlsHeader.propTypes = {
+	statusPage: PropTypes.object,
+	deleteStatusPage: PropTypes.func,
+	isDeleting: PropTypes.bool,
+};
+
 export default ControlsHeader;
diff --git a/Client/src/Pages/StatusPage/Status/Components/StatusBar/index.jsx b/Client/src/Pages/StatusPage/Status/Components/StatusBar/index.jsx
index ec457ddb1..1898ff3d7 100644
--- a/Client/src/Pages/StatusPage/Status/Components/StatusBar/index.jsx
+++ b/Client/src/Pages/StatusPage/Status/Components/StatusBar/index.jsx
@@ -70,5 +70,5 @@ const StatusBar = ({ monitors }) => {
 export default StatusBar;
 
 StatusBar.propTypes = {
-	status: PropTypes.object.isRequired,
+	status: PropTypes.object,
 };