Skip to content

Fix ModifyOriginalResponseAsync rejecting null content with attachments - #3279

Merged
Misha-133 merged 1 commit into
discord-net:devfrom
Zont1k:fix-modify-original-response-null-content
Aug 11, 2026
Merged

Misha-133 merged 1 commit into
discord-net:devfrom
Zont1k:fix-modify-original-response-null-content

Conversation

@Zont1k

@Zont1k Zont1k commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Description

Allows setting Content = null in ModifyOriginalResponseAsync when Attachments are provided. Previously the Preconditions.NotNullOrEmpty check would throw even when the user intended to send only attachments without text.

Changes

  • Added hasAttachments check in ModifyInteractionResponseAsync
  • Added hasAttachments check in ModifyFollowupMessageAsync

Related Issues

Allows setting Content to null when Attachments are provided,
fixing a precondition that incorrectly required non-empty content
even when the user intended to send only attachments.
@Misha-133
Misha-133 merged commit 9c78ee0 into discord-net:dev Aug 11, 2026
cgwhouse pushed a commit to cgwhouse/widen-bot that referenced this pull request Oct 9, 2026
Updated [Discord.Net](https://github.com/discord-net/Discord.Net) from
3.20.1 to 3.20.2.

<details>
<summary>Release notes</summary>

_Sourced from [Discord.Net's
releases](https://github.com/discord-net/Discord.Net/releases)._

## 3.20.2

## [3.20.2] - 2026-10-08

A small update that brings support for the new file upload component
file extension filter as well as some other new API things.

### Added
- #​3280 Add attachment flags and fix container spoiler state (d550c75)
- #​3282 Add IsSpoilerChannel flag (bdecf12)
- #​3286 Add file type filters support for file upload components #​3286
(d34a50e)

### Fixed
- #​3279 Fix ModifyOriginalResponseAsync rejecting null content with
attachments (9c78ee0)

### Misc
- #​3283 Add audio recording sample (bd4a42c)
- #​3284 Give some extra info when no selectmenu options are given
(911efb4)
- #​3285 Change default attachment size limit to 20MiB (247b1e5)

## New Contributors
* @​Zont1k made their first contribution in
discord-net/Discord.Net#3279

**Full Changelog**:
discord-net/Discord.Net@3.20.1...3.20.2

Commits viewable in [compare
view](discord-net/Discord.Net@3.20.1...3.20.2).
</details>

[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=Discord.Net&package-manager=nuget&previous-version=3.20.1&new-version=3.20.2)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: You cannot modify the text of a message to null using ModifyOriginalResponseAsync

2 participants