Merge pull request #1587 from DMJain/develop

#1584 Removed Checkboxes for Notification : SMS & Multi Email
This commit is contained in:
Alexander Holliday
2025-01-20 11:03:57 -08:00
committed by GitHub

View File

@@ -376,14 +376,15 @@ const Configure = () => {
</Box>
<Stack gap={theme.spacing(6)}>
<Typography component="p">When there is a new incident,</Typography>
<Checkbox
{/* {Leaving components commented for future funtionality implimentation} */}
{/* <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,15 +396,15 @@ const Configure = () => {
value={user?.email}
onChange={(event) => handleChange(event)}
/>
<Checkbox
{/* <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(
/> */}
{/* {monitor?.notifications?.some(
(notification) => notification.type === "emails"
) ? (
<Box mx={theme.spacing(16)}>
@@ -420,7 +421,7 @@ const Configure = () => {
</Box>
) : (
""
)}
)} */}
</Stack>
</ConfigBox>
<ConfigBox>