matt/mailship
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: guard deleteAlert and publishAlert against undefined relay

Browse source 
NOTIFIER_RELAY is string | undefined after the earlier guard fix.
deleteAlert and publishAlert passed it directly to publish() which
expects string[]. Add early-return guards to both functions.

Fixes the 2 new TS2322 errors introduced by the previous commit.
This commit is contained in:
Agent 2026-08-24 18:22:46 -04:00
parent 57f7d94669
commit 4e6030f3c2
1 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
web/src/main.ts
View file

@ -144,6 +144,8 @@ const loadAlerts = async () => {
}
const deleteAlert = async (alert: Alert) => {
if (!NOTIFIER_RELAY) return
if (confirm("Are you sure you want to delete this alert?")) {
state.update(assoc('alertsLoading', true))
@ -207,8 +209,11 @@ export const makeAlert = async ({freq, time, email, feeds, secret}: AlertParams)
)
}
export const publishAlert = async (params: AlertParams) =>
publish({event: await makeAlert(params), relays: [NOTIFIER_RELAY]})
export const publishAlert = async (params: AlertParams) => {
if (!NOTIFIER_RELAY) return
await publish({event: await makeAlert(params), relays: [NOTIFIER_RELAY]})
}
// Components