#1584 removed checkbox for incident notification: SMS, Multi Email

This commit is contained in:
Darshan Jain committed 2025-01-19 23:30:08 +05:30
1 parent 7abc2280af
commit 4d2c59b30c
1 file changed
-16
@@ -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"
) ? (