Skip to content

Features/291 missing flags property on edit webhook message#292

Merged
BrammyS merged 3 commits intodevfrom
features/291-missing-flags-property-on-edit-webhook-message
Aug 17, 2025
Merged

Features/291 missing flags property on edit webhook message#292
BrammyS merged 3 commits intodevfrom
features/291-missing-flags-property-on-edit-webhook-message

Conversation

@BrammyS
Copy link
Copy Markdown
Member

@BrammyS BrammyS commented Aug 17, 2025

Description

Closes #291

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@BrammyS BrammyS requested a review from Copilot August 17, 2025 08:53
@BrammyS BrammyS self-assigned this Aug 17, 2025
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for message flags in webhook message editing functionality, addressing issue #291. The changes enable the passing of Discord message flags (such as SUPPRESS_EMBEDS and IS_COMPONENTS_V2) when editing webhook messages through the interaction controller.

  • Added Flags property to the DiscordEditWebhookMessage record
  • Updated the interaction controller to pass through the flags from response data

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
DiscordEditWebhookMessage.cs Added Flags property with appropriate JSON serialization to support message flags
DiscordInteractionController.cs Updated webhook message editing to include flags from response data

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@BrammyS BrammyS merged commit a30ae51 into dev Aug 17, 2025
1 check passed
@BrammyS BrammyS deleted the features/291-missing-flags-property-on-edit-webhook-message branch August 17, 2025 08:54
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.

2 participants