From 8349cd7e1eb92e9a3baff68082c30a7b43e8e9b7 Mon Sep 17 00:00:00 2001 From: th0mk Date: Sat, 10 Jul 2021 16:40:52 +0200 Subject: [PATCH 1/3] fix: Change embed description max length to 4096 (#1886) * Update max embed description length (fixes #1881) * Update unit tests for new embed builder length --- src/Discord.Net.Core/Entities/Messages/EmbedBuilder.cs | 2 +- test/Discord.Net.Tests.Unit/EmbedBuilderTests.cs | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/Discord.Net.Core/Entities/Messages/EmbedBuilder.cs b/src/Discord.Net.Core/Entities/Messages/EmbedBuilder.cs index f1238ddcf..89aaf5fde 100644 --- a/src/Discord.Net.Core/Entities/Messages/EmbedBuilder.cs +++ b/src/Discord.Net.Core/Entities/Messages/EmbedBuilder.cs @@ -27,7 +27,7 @@ namespace Discord /// /// Returns the maximum length of description allowed by Discord. /// - public const int MaxDescriptionLength = 2048; + public const int MaxDescriptionLength = 4096; /// /// Returns the maximum length of total characters allowed by Discord. /// diff --git a/test/Discord.Net.Tests.Unit/EmbedBuilderTests.cs b/test/Discord.Net.Tests.Unit/EmbedBuilderTests.cs index 6cfdc83b2..da21afee1 100644 --- a/test/Discord.Net.Tests.Unit/EmbedBuilderTests.cs +++ b/test/Discord.Net.Tests.Unit/EmbedBuilderTests.cs @@ -126,7 +126,7 @@ namespace Discord { IEnumerable GetInvalid() { - yield return new string('a', 2049); + yield return new string('a', 4097); } foreach (var description in GetInvalid()) { @@ -149,7 +149,7 @@ namespace Discord { yield return string.Empty; yield return null; - yield return new string('a', 2048); + yield return new string('a', 4096); } foreach (var description in GetValid()) { From c5b4b645b8eeea9698453be0602ad07dba9dabf8 Mon Sep 17 00:00:00 2001 From: Wardog <49387986+wrdg@users.noreply.github.com> Date: Sat, 10 Jul 2021 09:42:45 -0500 Subject: [PATCH 2/3] feature: Add Name property to Teams (#1879) --- src/Discord.Net.Core/Entities/Teams/ITeam.cs | 4 ++++ src/Discord.Net.Rest/API/Common/Team.cs | 2 ++ src/Discord.Net.Rest/Entities/Teams/RestTeam.cs | 3 +++ 3 files changed, 9 insertions(+) diff --git a/src/Discord.Net.Core/Entities/Teams/ITeam.cs b/src/Discord.Net.Core/Entities/Teams/ITeam.cs index 5ef3e4253..b6e3d987b 100644 --- a/src/Discord.Net.Core/Entities/Teams/ITeam.cs +++ b/src/Discord.Net.Core/Entities/Teams/ITeam.cs @@ -20,6 +20,10 @@ namespace Discord /// IReadOnlyList TeamMembers { get; } /// + /// Gets the name of this team. + /// + string Name { get; } + /// /// Gets the user identifier that owns this team. /// ulong OwnerUserId { get; } diff --git a/src/Discord.Net.Rest/API/Common/Team.cs b/src/Discord.Net.Rest/API/Common/Team.cs index 4910f43f7..852368522 100644 --- a/src/Discord.Net.Rest/API/Common/Team.cs +++ b/src/Discord.Net.Rest/API/Common/Team.cs @@ -11,6 +11,8 @@ namespace Discord.API public ulong Id { get; set; } [JsonProperty("members")] public TeamMember[] TeamMembers { get; set; } + [JsonProperty("name")] + public string Name { get; set; } [JsonProperty("owner_user_id")] public ulong OwnerUserId { get; set; } } diff --git a/src/Discord.Net.Rest/Entities/Teams/RestTeam.cs b/src/Discord.Net.Rest/Entities/Teams/RestTeam.cs index 2343f8b5d..43c9417cc 100644 --- a/src/Discord.Net.Rest/Entities/Teams/RestTeam.cs +++ b/src/Discord.Net.Rest/Entities/Teams/RestTeam.cs @@ -12,6 +12,8 @@ namespace Discord.Rest /// public IReadOnlyList TeamMembers { get; private set; } /// + public string Name { get; private set; } + /// public ulong OwnerUserId { get; private set; } private string _iconId; @@ -30,6 +32,7 @@ namespace Discord.Rest { if (model.Icon.IsSpecified) _iconId = model.Icon.Value; + Name = model.Name; OwnerUserId = model.OwnerUserId; TeamMembers = model.TeamMembers.Select(x => new RestTeamMember(Discord, x)).ToImmutableArray(); } From c20086158572acf5fbc3c795769d12b39b127482 Mon Sep 17 00:00:00 2001 From: Quin Lynch <49576606+quinchs@users.noreply.github.com> Date: Sat, 10 Jul 2021 11:44:32 -0300 Subject: [PATCH 3/3] fix: Add default avatar to WithAuthor extension (#1890) Adds `?? user.GetDefaultAvatarUrl()` to the avatar icon field. --- src/Discord.Net.Core/Extensions/EmbedBuilderExtensions.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Discord.Net.Core/Extensions/EmbedBuilderExtensions.cs b/src/Discord.Net.Core/Extensions/EmbedBuilderExtensions.cs index a3b8ddd5b..c05df7cb7 100644 --- a/src/Discord.Net.Core/Extensions/EmbedBuilderExtensions.cs +++ b/src/Discord.Net.Core/Extensions/EmbedBuilderExtensions.cs @@ -27,7 +27,7 @@ namespace Discord /// Fills the embed author field with the provided user's full username and avatar URL. public static EmbedBuilder WithAuthor(this EmbedBuilder builder, IUser user) => - builder.WithAuthor($"{user.Username}#{user.Discriminator}", user.GetAvatarUrl()); + builder.WithAuthor($"{user.Username}#{user.Discriminator}", user.GetAvatarUrl() ?? user.GetDefaultAvatarUrl()); /// Converts a object to a . /// The embed type is not .