fix(discord): 🐛 Fixed an issue where the Icon in the discord notifica…

…tions wouldn't apply
Ombi-app · Nov 1, 2023 · 32da949 · 32da949
1 parent 7bc915c
commit 32da949
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/src/Ombi.Api.Discord/Models/DiscordWebhookBody.cs b/src/Ombi.Api.Discord/Models/DiscordWebhookBody.cs
@@ -7,6 +7,7 @@ public class DiscordWebhookBody
     {
         public string content { get; set; }
         public string username { get; set; }
+        public string avatar_url { get; set; }
         public List<DiscordEmbeds> embeds { get; set; }
     }
 
@@ -32,7 +33,6 @@ public class DiscordAuthor
     {
         public string name { get; set; }
         public string url { get; set; }
-        public string iconurl { get; set; }
     }
 
     public class DiscordField

diff --git a/src/Ombi.Notifications/Agents/DiscordNotification.cs b/src/Ombi.Notifications/Agents/DiscordNotification.cs
@@ -108,6 +108,7 @@ protected override async Task Send(NotificationMessage model, DiscordNotificatio
                 {
                     content = model.Message,
                     username = settings.Username ?? "Ombi",
+                    avatar_url = settings.Icon.HasValue() ? settings.Icon : string.Empty
                 };
 
                 var fields = new List<DiscordField>();
@@ -157,10 +158,7 @@ protected override async Task Send(NotificationMessage model, DiscordNotificatio
                     }
                 }
 
-                var author = new DiscordAuthor
-                {
-                    iconurl = settings.Icon.HasValue() ? settings.Icon : string.Empty
-                };
+                var author = new DiscordAuthor();
 
                 if (model.Data.TryGetValue("ApplicationUrl", out var appUrl))
                 {