From 2e869310b88d6c472335fe5a6860837fcdf2e048 Mon Sep 17 00:00:00 2001
From: Amol <meamolvyas@gmail.com>
Date: Tue, 1 Jul 2025 22:44:42 +0530
Subject: [PATCH] feat: disable send test notification when 0 notification
 configured

---
 .../MonitorDetailsControlHeader/index.jsx     | 61 +++++++++++++++----
 1 file changed, 48 insertions(+), 13 deletions(-)

diff --git a/client/src/Components/MonitorDetailsControlHeader/index.jsx b/client/src/Components/MonitorDetailsControlHeader/index.jsx
index 1d253e3d5..9a76af551 100644
--- a/client/src/Components/MonitorDetailsControlHeader/index.jsx
+++ b/client/src/Components/MonitorDetailsControlHeader/index.jsx
@@ -2,6 +2,7 @@ import Stack from "@mui/material/Stack";
 import Status from "./status";
 import Skeleton from "./skeleton";
 import Button from "@mui/material/Button";
+import { Tooltip } from "@mui/material";
 import SettingsOutlinedIcon from "@mui/icons-material/SettingsOutlined";
 import PauseOutlinedIcon from "@mui/icons-material/PauseOutlined";
 import PlayArrowOutlinedIcon from "@mui/icons-material/PlayArrowOutlined";
@@ -40,6 +41,8 @@ const MonitorDetailsControlHeader = ({
 	const { t } = useTranslation();
 	const [pauseMonitor, isPausing, error] = usePauseMonitor();
 
+	const isTestNotificationsDisabled = monitor?.notifications?.length === 0;
+
 	// const [isSending, emailError, sendTestEmail] = useSendTestEmail();
 
 	const [testAllNotifications, isSending, errorAllNotifications] =
@@ -60,20 +63,52 @@ const MonitorDetailsControlHeader = ({
 				direction="row"
 				gap={theme.spacing(2)}
 			>
-				<Button
-					variant="contained"
-					color="secondary"
-					loading={isSending}
-					startIcon={<EmailIcon />}
-					onClick={() => {
-						testAllNotifications({ monitorId: monitor?._id });
-					}}
-					sx={{
-						whiteSpace: "nowrap",
-					}}
+				<Tooltip
+					key={monitor?._id}
+					placement="bottom"
+					title={
+						isTestNotificationsDisabled
+							? t(
+									'There are no notifications setup for this monitor. You need to add one by clicking "Configure" button'
+								)
+							: ""
+					}
 				>
-					{t("sendTestNotifications")}
-				</Button>
+					{isTestNotificationsDisabled ? (
+						<span>
+							<Button
+								variant="contained"
+								color="secondary"
+								loading={isSending}
+								startIcon={<EmailIcon />}
+								disabled
+								onClick={() => {
+									testAllNotifications({ monitorId: monitor?._id });
+								}}
+								sx={{
+									whiteSpace: "nowrap",
+								}}
+							>
+								{t("sendTestNotifications")}
+							</Button>
+						</span>
+					) : (
+						<Button
+							variant="contained"
+							color="secondary"
+							loading={isSending}
+							startIcon={<EmailIcon />}
+							onClick={() => {
+								testAllNotifications({ monitorId: monitor?._id });
+							}}
+							sx={{
+								whiteSpace: "nowrap",
+							}}
+						>
+							{t("sendTestNotifications")}
+						</Button>
+					)}
+				</Tooltip>
 				<Button
 					variant="contained"
 					color="secondary"