fix(notifications): 🐛 Do not notify user upon auto approval of a TV s…

…how (#4432) Fixes #4431
Ombi-app · Dec 8, 2021 · 3ad3bdd · 3ad3bdd
1 parent 8943a97
commit 3ad3bdd
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/src/Ombi.Core/Engine/TvRequestEngine.cs b/src/Ombi.Core/Engine/TvRequestEngine.cs
@@ -710,7 +710,11 @@ public async Task<RequestEngineResult> ApproveChildRequest(int id)
 
             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 @@ private async Task<RequestEngineResult> ProcessSendingShow(ChildRequests model)
             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)
                 {