Browse Source

Make ModifyWebhookMessageParams.Content Optional<string>

pull/1753/head
Desmont 4 years ago
parent
commit
f9378d5a30
2 changed files with 2 additions and 2 deletions
  1. +1
    -1
      src/Discord.Net.Rest/API/Rest/ModifyWebhookMessageParams.cs
  2. +1
    -1
      src/Discord.Net.Rest/DiscordRestApiClient.cs

+ 1
- 1
src/Discord.Net.Rest/API/Rest/ModifyWebhookMessageParams.cs View File

@@ -7,7 +7,7 @@ namespace Discord.API.Rest
internal class ModifyWebhookMessageParams
{
[JsonProperty("content")]
public string Content { get; set; }
public Optional<string> Content { get; set; }
[JsonProperty("embeds")]
public Optional<Embed[]> Embeds { get; set; }
[JsonProperty("allowed_mentions")]


+ 1
- 1
src/Discord.Net.Rest/DiscordRestApiClient.cs View File

@@ -537,7 +537,7 @@ namespace Discord.API

if (args.Embeds.IsSpecified)
Preconditions.AtMost(args.Embeds.Value.Length, 10, nameof(args.Embeds), "A max of 10 Embeds are allowed.");
if (args.Content?.Length > DiscordConfig.MaxMessageSize)
if (args.Content.IsSpecified && args.Content.Value.Length > DiscordConfig.MaxMessageSize)
throw new ArgumentException(message: $"Message content is too long, length must be less or equal to {DiscordConfig.MaxMessageSize}.", paramName: nameof(args.Content));
options = RequestOptions.CreateOrClone(options);



Loading…
Cancel
Save