Add Enable/Disable options for webhooks (#283)

jellyfin · Sep 9, 2024 · 4a50fa2 · 4a50fa2
1 parent aa60e5a
commit 4a50fa2
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 1 deletion.
diff --git a/Jellyfin.Plugin.Webhook/Configuration/Web/config.html b/Jellyfin.Plugin.Webhook/Configuration/Web/config.html
@@ -76,6 +76,13 @@ <h2 class="sectionTitle">Webhook</h2>
             <input is="emby-input" type="text" data-name="txtWebhookUri" required="required" label="Webhook Url:"/>
             <span>The webhook destination url</span>
         </div>
+        <div class="inputContainer">
+            <label>Status:</label>
+            <label class="checkboxContainer">
+                <input is="emby-checkbox" type="checkbox" data-name="chkEnableWebhook"/>
+                <span>Enable</span>
+            </label>
+        </div>
         <div>
             <label>Notification Type:</label>
             <div data-name="notificationTypeContainer">

diff --git a/Jellyfin.Plugin.Webhook/Configuration/Web/config.js b/Jellyfin.Plugin.Webhook/Configuration/Web/config.js
@@ -164,6 +164,7 @@ export default function (view) {
                 element.querySelector("[data-name=chkSendAllProperties]").checked = config.SendAllProperties || false;
                 element.querySelector("[data-name=chkTrimWhitespace]").checked = config.TrimWhitespace || false;
                 element.querySelector("[data-name=chkSkipEmptyMessageBody]").checked = config.SkipEmptyMessageBody || false;
+                element.querySelector("[data-name=chkEnableWebhook]").checked = config.EnableWebhook !== undefined ? config.EnableWebhook : true;
                 element.querySelector("[data-name=txtTemplate]").value = Webhook.atou(config.Template || "");
 
                 const notificationTypeContainer = element.querySelector("[data-name=notificationTypeContainer]");
@@ -187,6 +188,7 @@ export default function (view) {
                 config.SendAllProperties = element.querySelector("[data-name=chkSendAllProperties]").checked || false;
                 config.TrimWhitespace = element.querySelector("[data-name=chkTrimWhitespace]").checked || false;
                 config.SkipEmptyMessageBody = element.querySelector("[data-name=chkSkipEmptyMessageBody]").checked || false;
+                config.EnableWebhook = element.querySelector("[data-name=chkEnableWebhook]").checked;
                 config.Template = Webhook.utoa(element.querySelector("[data-name=txtTemplate]").value || "");
 
                 config.NotificationTypes = [];

diff --git a/Jellyfin.Plugin.Webhook/Destinations/BaseOption.cs b/Jellyfin.Plugin.Webhook/Destinations/BaseOption.cs
@@ -82,6 +82,11 @@ public abstract class BaseOption
     /// </summary>
     public bool SkipEmptyMessageBody { get; set; }
 
+    /// <summary>
+    /// Gets or sets a value indicating whether to Enable or Disable Webhook.
+    /// </summary>
+    public bool EnableWebhook { get; set; } = true;
+
     /// <summary>
     /// Gets or sets the handlebars template.
     /// </summary>

diff --git a/Jellyfin.Plugin.Webhook/WebhookSender.cs b/Jellyfin.Plugin.Webhook/WebhookSender.cs
@@ -182,7 +182,7 @@ private static bool NotifyOnItem<T>(T baseOptions, Type? itemType)
     private async Task SendNotification<T>(IWebhookClient<T> webhookClient, T option, Dictionary<string, object> itemData, Type? itemType)
         where T : BaseOption
     {
-        if (NotifyOnItem(option, itemType))
+        if (option.EnableWebhook && NotifyOnItem(option, itemType))
         {
             try
             {