From c28ec13524a0f788030c005202c54d4d3114bfae Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 12 Feb 2026 16:27:16 +0000 Subject: [PATCH] Inline thumbnail URL logic directly into attributes - Remove GetThumbnailUrl method from YouTubeVideoEmbedProjection - Remove local variables from template - Put coalescing logic directly in src and data-canonical-url attributes Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com> --- .../Discord/Data/Embeds/YouTubeVideoEmbedProjection.cs | 3 --- .../Exporting/MessageGroupTemplate.cshtml | 6 +----- 2 files changed, 1 insertion(+), 8 deletions(-) diff --git a/DiscordChatExporter.Core/Discord/Data/Embeds/YouTubeVideoEmbedProjection.cs b/DiscordChatExporter.Core/Discord/Data/Embeds/YouTubeVideoEmbedProjection.cs index 01abaf58..f6d095d2 100644 --- a/DiscordChatExporter.Core/Discord/Data/Embeds/YouTubeVideoEmbedProjection.cs +++ b/DiscordChatExporter.Core/Discord/Data/Embeds/YouTubeVideoEmbedProjection.cs @@ -6,9 +6,6 @@ public partial record YouTubeVideoEmbedProjection(string VideoId) // Using hqdefault.jpg which is guaranteed to exist for all YouTube videos public string ThumbnailUrl { get; } = $"https://i.ytimg.com/vi/{VideoId}/hqdefault.jpg"; - - public string GetThumbnailUrl(Embed embed) => - embed.Thumbnail?.ProxyUrl ?? embed.Thumbnail?.Url ?? ThumbnailUrl; } public partial record YouTubeVideoEmbedProjection diff --git a/DiscordChatExporter.Core/Exporting/MessageGroupTemplate.cshtml b/DiscordChatExporter.Core/Exporting/MessageGroupTemplate.cshtml index bfd3b6ba..641c2ada 100644 --- a/DiscordChatExporter.Core/Exporting/MessageGroupTemplate.cshtml +++ b/DiscordChatExporter.Core/Exporting/MessageGroupTemplate.cshtml @@ -427,11 +427,7 @@ @* Video thumbnail *@
- @{ - var thumbnailUrl = youTubeVideoEmbed.GetThumbnailUrl(embed); - var thumbnailCanonicalUrl = embed.Thumbnail?.Url ?? youTubeVideoEmbed.ThumbnailUrl; - } - YouTube video thumbnail + YouTube video thumbnail