From 4d2c59b30caf242212abb92693ab229e3a8ff9dc Mon Sep 17 00:00:00 2001
From: Darshan Jain <darshan.j098@gmail.com>
Date: Sun, 19 Jan 2025 23:30:08 +0530
Subject: [PATCH 1/4] #1584 removed checkbox for incident notification: SMS,
 Multi Email

---
 Client/src/Pages/Uptime/Configure/index.jsx | 16 ----------------
 1 file changed, 16 deletions(-)

diff --git a/Client/src/Pages/Uptime/Configure/index.jsx b/Client/src/Pages/Uptime/Configure/index.jsx
index 6cef97e0f..bb405cc68 100644
--- a/Client/src/Pages/Uptime/Configure/index.jsx
+++ b/Client/src/Pages/Uptime/Configure/index.jsx
@@ -376,14 +376,6 @@ const Configure = () => {
 							</Box>
 							<Stack gap={theme.spacing(6)}>
 								<Typography component="p">When there is a new incident,</Typography>
-								<Checkbox
-									id="notify-sms"
-									label="Notify via SMS (coming soon)"
-									isChecked={false}
-									value=""
-									onChange={() => logger.warn("disabled")}
-									isDisabled={true}
-								/>
 								<Checkbox
 									id="notify-email-default"
 									label={`Notify via email (to ${user.email})`}
@@ -395,14 +387,6 @@ const Configure = () => {
 									value={user?.email}
 									onChange={(event) => handleChange(event)}
 								/>
-								<Checkbox
-									id="notify-email"
-									label="Also notify via email to multiple addresses (coming soon)"
-									isChecked={false}
-									value=""
-									onChange={() => logger.warn("disabled")}
-									isDisabled={true}
-								/>
 								{monitor?.notifications?.some(
 									(notification) => notification.type === "emails"
 								) ? (

From 6ed8f93688569dd7481a3039299e7068085fd231 Mon Sep 17 00:00:00 2001
From: Darshan Jain <darshan.j098@gmail.com>
Date: Sun, 19 Jan 2025 23:35:01 +0530
Subject: [PATCH 2/4] #1584 removed conditional input box for multi email

---
 Client/src/Pages/Uptime/Configure/index.jsx | 18 ------------------
 1 file changed, 18 deletions(-)

diff --git a/Client/src/Pages/Uptime/Configure/index.jsx b/Client/src/Pages/Uptime/Configure/index.jsx
index bb405cc68..1b2e448ac 100644
--- a/Client/src/Pages/Uptime/Configure/index.jsx
+++ b/Client/src/Pages/Uptime/Configure/index.jsx
@@ -387,24 +387,6 @@ const Configure = () => {
 									value={user?.email}
 									onChange={(event) => handleChange(event)}
 								/>
-								{monitor?.notifications?.some(
-									(notification) => notification.type === "emails"
-								) ? (
-									<Box mx={theme.spacing(16)}>
-										<TextInput
-											id="notify-email-list"
-											type="text"
-											placeholder="name@gmail.com"
-											value=""
-											onChange={() => logger.warn("disabled")}
-										/>
-										<Typography mt={theme.spacing(4)}>
-											You can separate multiple emails with a comma
-										</Typography>
-									</Box>
-								) : (
-									""
-								)}
 							</Stack>
 						</ConfigBox>
 						<ConfigBox>

From d8b1d63faa64d9024566d3bf369381a3e4ec21bc Mon Sep 17 00:00:00 2001
From: Darshan Jain <darshan.j098@gmail.com>
Date: Mon, 20 Jan 2025 13:26:29 +0530
Subject: [PATCH 3/4] commenting out checkboxs and input box instead of
 deleting for future implementations

---
 Client/src/Pages/Uptime/Configure/index.jsx | 34 +++++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/Client/src/Pages/Uptime/Configure/index.jsx b/Client/src/Pages/Uptime/Configure/index.jsx
index 1b2e448ac..d698e6624 100644
--- a/Client/src/Pages/Uptime/Configure/index.jsx
+++ b/Client/src/Pages/Uptime/Configure/index.jsx
@@ -376,6 +376,14 @@ const Configure = () => {
 							</Box>
 							<Stack gap={theme.spacing(6)}>
 								<Typography component="p">When there is a new incident,</Typography>
+								{/* <Checkbox
+									id="notify-sms"
+									label="Notify via SMS (coming soon)"
+									isChecked={false}
+									value=""
+									onChange={() => logger.warn("disabled")}
+									isDisabled={true}
+								/> */}
 								<Checkbox
 									id="notify-email-default"
 									label={`Notify via email (to ${user.email})`}
@@ -387,6 +395,32 @@ const Configure = () => {
 									value={user?.email}
 									onChange={(event) => handleChange(event)}
 								/>
+								{/* <Checkbox
+									id="notify-email"
+									label="Also notify via email to multiple addresses (coming soon)"
+									isChecked={false}
+									value=""
+									onChange={() => logger.warn("disabled")}
+									isDisabled={true}
+								/> */}
+								{/* {monitor?.notifications?.some(
+									(notification) => notification.type === "emails"
+								) ? (
+									<Box mx={theme.spacing(16)}>
+										<TextInput
+											id="notify-email-list"
+											type="text"
+											placeholder="name@gmail.com"
+											value=""
+											onChange={() => logger.warn("disabled")}
+										/>
+										<Typography mt={theme.spacing(4)}>
+											You can separate multiple emails with a comma
+										</Typography>
+									</Box>
+								) : (
+									""
+								)} */}
 							</Stack>
 						</ConfigBox>
 						<ConfigBox>

From ed9a89b52604bc239cbd52074ca4a529d113b9cb Mon Sep 17 00:00:00 2001
From: Darshan Jain <darshan.j098@gmail.com>
Date: Mon, 20 Jan 2025 16:36:50 +0530
Subject: [PATCH 4/4] #1584 comments

---
 Client/src/Pages/Uptime/Configure/index.jsx | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Client/src/Pages/Uptime/Configure/index.jsx b/Client/src/Pages/Uptime/Configure/index.jsx
index d698e6624..9f244d0a2 100644
--- a/Client/src/Pages/Uptime/Configure/index.jsx
+++ b/Client/src/Pages/Uptime/Configure/index.jsx
@@ -376,6 +376,7 @@ const Configure = () => {
 							</Box>
 							<Stack gap={theme.spacing(6)}>
 								<Typography component="p">When there is a new incident,</Typography>
+								{/* {Leaving components commented for future funtionality implimentation} */}
 								{/* <Checkbox
 									id="notify-sms"
 									label="Notify via SMS (coming soon)"