From b6803717462963104648fc61da62e7717a95090d Mon Sep 17 00:00:00 2001 From: Clemens Wolff Date: Sat, 7 May 2022 11:00:57 -0400 Subject: [PATCH] Make apprise notification title configurable in UI --- server/notification-providers/apprise.js | 11 ++--------- src/components/notifications/Apprise.vue | 3 +++ 2 files changed, 5 insertions(+), 9 deletions(-) diff --git a/server/notification-providers/apprise.js b/server/notification-providers/apprise.js index acbc05673..887afbf55 100644 --- a/server/notification-providers/apprise.js +++ b/server/notification-providers/apprise.js @@ -1,22 +1,15 @@ const NotificationProvider = require("./notification-provider"); const childProcess = require("child_process"); -/** - * If you use an apprise backend that requires the notification title to - * be set (such as for example messaging a Zulip Stream), you can use this - * environment variable to configure the title. - */ -const { APPRISE_NOTIFICATION_TITLE } = process.env; - class Apprise extends NotificationProvider { name = "apprise"; async send(notification, msg, monitorJSON = null, heartbeatJSON = null) { const args = [ "-vv", "-b", msg, notification.appriseURL ]; - if (APPRISE_NOTIFICATION_TITLE) { + if (notification.title) { args.push("-t"); - args.push(APPRISE_NOTIFICATION_TITLE); + args.push(notification.title); } const s = childProcess.spawnSync("apprise", args); diff --git a/src/components/notifications/Apprise.vue b/src/components/notifications/Apprise.vue index c10e23cf9..7432554c8 100644 --- a/src/components/notifications/Apprise.vue +++ b/src/components/notifications/Apprise.vue @@ -8,6 +8,9 @@ https://github.com/caronc/apprise/wiki#notification-services + + +