fix(notifications): 🐛 Do not notify user upon auto approval of a TV show (#4432)

Fixes #4431
3 years ago · 3ad3bddd83
parent 8943a978bf
commit 3ad3bddd83
1 changed files with 10 additions and 2 deletions
--- a/src/Ombi.Core/Engine/TvRequestEngine.cs
+++ b/src/Ombi.Core/Engine/TvRequestEngine.cs
@ -710,7 +710,11 @@ namespace Ombi.Core.Engine

            if (request.Approved)
            {
-                await NotificationHelper.Notify(request, NotificationType.RequestApproved);
+                var canNotify = await RunSpecificRule(request, SpecificRules.CanSendNotification, string.Empty);
+                if (canNotify.Success)
+                {
+                    await NotificationHelper.Notify(request, NotificationType.RequestApproved);
+                }
                // Autosend
                await TvSender.Send(request);
            }
@ -957,7 +961,11 @@ namespace Ombi.Core.Engine
            if (model.Approved)
            {
                // Autosend
-                await NotificationHelper.Notify(model, NotificationType.RequestApproved);
+                var canNotify = await RunSpecificRule(model, SpecificRules.CanSendNotification, string.Empty);
+                if (canNotify.Success)
+                {
+                    await NotificationHelper.Notify(model, NotificationType.RequestApproved);
+                }
                var result = await TvSender.Send(model);
                if (result.Success)
                {