fixed build and added logging

src/Ombi.Notifications/Agents/DiscordNotification.cs

@@ -18,7 +18,10 @@ namespace Ombi.Notifications.Agents
 {
     public class DiscordNotification : BaseNotification<DiscordNotificationSettings>, IDiscordNotification
     {
-        public DiscordNotification(IDiscordApi api, ISettingsService<DiscordNotificationSettings> sn, ILogger<DiscordNotification> log, INotificationTemplatesRepository r, IMovieRequestRepository m, ITvRequestRepository t, ISettingsService<CustomizationSettings> s) : base(sn, r, m, t,s)
+        public DiscordNotification(IDiscordApi api, ISettingsService<DiscordNotificationSettings> sn,
+                                   ILogger<DiscordNotification> log, INotificationTemplatesRepository r,
+                                   IMovieRequestRepository m, ITvRequestRepository t, ISettingsService<CustomizationSettings> s)
+            : base(sn, r, m, t,s,log)
         {
             Api = api;
             Logger = log;

src/Ombi.Notifications/Agents/EmailNotification.cs

@@ -19,7 +19,7 @@ namespace Ombi.Notifications.Agents
     public class EmailNotification : BaseNotification<EmailNotificationSettings>, IEmailNotification
     {
         public EmailNotification(ISettingsService<EmailNotificationSettings> settings, INotificationTemplatesRepository r, IMovieRequestRepository m, ITvRequestRepository t, IEmailProvider prov, ISettingsService<CustomizationSettings> c,
-            ILogger<EmailNotification> log) : base(settings, r, m, t, c)
+            ILogger<EmailNotification> log) : base(settings, r, m, t, c,log)
         {
             EmailProvider = prov;
             Logger = log;

src/Ombi.Notifications/Agents/MattermostNotification.cs

@@ -21,7 +21,7 @@ namespace Ombi.Notifications.Agents
     public class MattermostNotification : BaseNotification<MattermostNotificationSettings>, IMattermostNotification
     {
         public MattermostNotification(IMattermostApi api, ISettingsService<MattermostNotificationSettings> sn, ILogger<MattermostNotification> log, INotificationTemplatesRepository r, IMovieRequestRepository m, ITvRequestRepository t,
-            ISettingsService<CustomizationSettings> s) : base(sn, r, m, t,s)
+            ISettingsService<CustomizationSettings> s) : base(sn, r, m, t,s,log)
         {
             Api = api;
             Logger = log;

src/Ombi.Notifications/Agents/MobileNotification.cs

@@ -22,7 +22,7 @@ namespace Ombi.Notifications.Agents
     {
         public MobileNotification(IOneSignalApi api, ISettingsService<MobileNotificationSettings> sn, ILogger<MobileNotification> log, INotificationTemplatesRepository r,
             IMovieRequestRepository m, ITvRequestRepository t, ISettingsService<CustomizationSettings> s, IRepository<NotificationUserId> notification,
-            UserManager<OmbiUser> um) : base(sn, r, m, t, s)
+            UserManager<OmbiUser> um) : base(sn, r, m, t, s,log)
         {
             _api = api;
             _logger = log;

src/Ombi.Notifications/Agents/PushbulletNotification.cs

@@ -17,7 +17,7 @@ namespace Ombi.Notifications.Agents
     public class PushbulletNotification : BaseNotification<PushbulletSettings>, IPushbulletNotification
     {
         public PushbulletNotification(IPushbulletApi api, ISettingsService<PushbulletSettings> sn, ILogger<PushbulletNotification> log, INotificationTemplatesRepository r, IMovieRequestRepository m, ITvRequestRepository t,
-            ISettingsService<CustomizationSettings> s) : base(sn, r, m, t,s)
+            ISettingsService<CustomizationSettings> s) : base(sn, r, m, t,s,log)
         {
             Api = api;
             Logger = log;

src/Ombi.Notifications/Agents/PushoverNotification.cs

@@ -17,8 +17,8 @@ namespace Ombi.Notifications.Agents
 {
     public class PushoverNotification : BaseNotification<PushoverSettings>, IPushoverNotification
     {
-        public PushoverNotification(IPushoverApi api, ISettingsService<PushoverSettings> sn, ILogger<PushbulletNotification> log, INotificationTemplatesRepository r, IMovieRequestRepository m, ITvRequestRepository t,
-            ISettingsService<CustomizationSettings> s) : base(sn, r, m, t, s)
+        public PushoverNotification(IPushoverApi api, ISettingsService<PushoverSettings> sn, ILogger<PushoverNotification> log, INotificationTemplatesRepository r, IMovieRequestRepository m, ITvRequestRepository t,
+            ISettingsService<CustomizationSettings> s) : base(sn, r, m, t, s, log)
         {
             Api = api;
             Logger = log;
@@ -27,7 +27,7 @@ namespace Ombi.Notifications.Agents
         public override string NotificationName => "PushoverNotification";
 
         private IPushoverApi Api { get; }
-        private ILogger<PushbulletNotification> Logger { get; }
+        private ILogger<PushoverNotification> Logger { get; }
 
         protected override bool ValidateConfiguration(PushoverSettings settings)
         {

src/Ombi.Notifications/Agents/SlackNotification.cs

@@ -18,7 +18,7 @@ namespace Ombi.Notifications.Agents
     public class SlackNotification : BaseNotification<SlackNotificationSettings>, ISlackNotification
     {
         public SlackNotification(ISlackApi api, ISettingsService<SlackNotificationSettings> sn, ILogger<SlackNotification> log, INotificationTemplatesRepository r, IMovieRequestRepository m, ITvRequestRepository t,
-            ISettingsService<CustomizationSettings> s) : base(sn, r, m, t, s)
+            ISettingsService<CustomizationSettings> s) : base(sn, r, m, t, s, log)
         {
             Api = api;
             Logger = log;

src/Ombi.Notifications/Agents/TelegramNotification.cs

@@ -16,7 +16,9 @@ namespace Ombi.Notifications.Agents
 {
     public class TelegramNotification : BaseNotification<TelegramSettings>, ITelegramNotification
     {
-        public TelegramNotification(ITelegramApi api, ISettingsService<TelegramSettings> sn, ILogger<TelegramNotification> log, INotificationTemplatesRepository r, IMovieRequestRepository m, ITvRequestRepository t, ISettingsService<CustomizationSettings> s) : base(sn, r, m, t,s)
+        public TelegramNotification(ITelegramApi api, ISettingsService<TelegramSettings> sn, ILogger<TelegramNotification> log, 
+                                    INotificationTemplatesRepository r, IMovieRequestRepository m, 
+                                    ITvRequestRepository t, ISettingsService<CustomizationSettings> s) : base(sn, r, m, t,s,log)
         {
             Api = api;
             Logger = log;

src/Ombi.Notifications/Interfaces/BaseNotification.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Threading.Tasks;
 using Microsoft.EntityFrameworkCore;
+using Microsoft.Extensions.Logging;
 using Ombi.Core.Settings;
 using Ombi.Helpers;
 using Ombi.Notifications.Exceptions;
@@ -16,7 +17,7 @@ namespace Ombi.Notifications.Interfaces
     public abstract class BaseNotification<T> : INotification where T : Settings.Settings.Models.Settings, new()
     {
         protected BaseNotification(ISettingsService<T> settings, INotificationTemplatesRepository templateRepo, IMovieRequestRepository movie, ITvRequestRepository tv,
-            ISettingsService<CustomizationSettings> customization)
+            ISettingsService<CustomizationSettings> customization, ILogger<BaseNotification<T>> log)
         {
             Settings = settings;
             TemplateRepository = templateRepo;
@@ -25,6 +26,7 @@ namespace Ombi.Notifications.Interfaces
             CustomizationSettings = customization;
             Settings.ClearCache();
             CustomizationSettings.ClearCache();
+            _log = log;
         }
         
         protected ISettingsService<T> Settings { get; }
@@ -33,6 +35,7 @@ namespace Ombi.Notifications.Interfaces
         protected ITvRequestRepository TvRepository { get; }
         protected CustomizationSettings Customization { get; set; }
         private ISettingsService<CustomizationSettings> CustomizationSettings { get; }
+        private readonly ILogger<BaseNotification<T>> _log;
 
 
         protected ChildRequests TvRequest { get; set; }
@@ -159,10 +162,13 @@ namespace Ombi.Notifications.Interfaces
             var curlys = new NotificationMessageCurlys();
             if (model.RequestType == RequestType.Movie)
             {
+                _log.LogDebug("Notification options: {@model}, Req: {@MovieRequest}, Settings: {@Customization}", model, MovieRequest, Customization);
+            
                 curlys.Setup(model, MovieRequest, Customization);
             }
             else
             {
+                _log.LogDebug("Notification options: {@model}, Req: {@TvRequest}, Settings: {@Customization}", model, TvRequest, Customization);
                 curlys.Setup(model, TvRequest, Customization);
             }
             var parsed = resolver.ParseMessage(template, curlys);

src/Ombi.Notifications/NotificationMessageCurlys.cs

@@ -11,16 +11,8 @@ namespace Ombi.Notifications
 {
     public class NotificationMessageCurlys
     {
-        public NotificationMessageCurlys(ILogger<NotificationMessageCurlys> log)
-        {
-            _log = log;
-        }
-
-        private readonly ILogger<NotificationMessageCurlys> _log;
-
         public void Setup(NotificationOptions opts, FullBaseRequest req, CustomizationSettings s)
         {
-            _log.LogDebug("Notification options: {@Opts}, Req: {@Req}, Settings: {@S}", opts, req, s);
             ApplicationUrl = (s?.ApplicationUrl.HasValue() ?? false) ? s.ApplicationUrl : string.Empty;
             ApplicationName = string.IsNullOrEmpty(s?.ApplicationName) ? "Ombi" : s?.ApplicationName;
             RequestedUser = string.IsNullOrEmpty(req.RequestedUser.Alias)
@@ -39,7 +31,6 @@ namespace Ombi.Notifications
         public void Setup(NotificationOptions opts, ChildRequests req, CustomizationSettings s)
         {
 
-            _log.LogDebug("Notification options: {@Opts}, Req: {@Req}, Settings: {@S}", opts, req, s);
            ApplicationUrl = (s?.ApplicationUrl.HasValue() ?? false) ? s.ApplicationUrl : string.Empty;
             ApplicationName = string.IsNullOrEmpty(s?.ApplicationName) ? "Ombi" : s?.ApplicationName;
             RequestedUser = string.IsNullOrEmpty(req.RequestedUser.Alias)