Skip to content

Updated "Message" and "Webhook" resource page and "Using Message Components" page with the new components v2 behaviour #7507

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
May 12, 2025

Conversation

PakjeBecel
Copy link
Contributor

@PakjeBecel PakjeBecel commented Apr 23, 2025

  • Updated various sections on the "Message" and "Webhook" resource page with the new components v2 behaviour.
  • Added a warning alert box on the "Using Message Components" page to make users aware of the IS_COMPONENTS_V2 message flag being irreverisble when set to a message.

Updated the section with the new components v2 behaviour.
Updated both sections with the new components v2 behaviour.
Added a warning alert box to make users aware of the "IS_COMPONENTS_V2" message flag being irreversible when set to a message.
@PakjeBecel PakjeBecel requested a review from a team as a code owner April 23, 2025 22:22
@PakjeBecel PakjeBecel requested review from colinloretz and removed request for a team April 23, 2025 22:22
@topi314
Copy link

topi314 commented Apr 24, 2025

https://discord.com/developers/docs/resources/webhook#edit-webhook-message-jsonform-params

the edit webhook docs are missing that you can also edit the flags of a webhook message (IS_COMPONENTS_V2 & SUPPRESS_EMBEDS)

I also notice that the IS_COMPONENTS_V2 is missing from the docs here:
https://discord.com/developers/docs/resources/webhook#execute-webhook-jsonform-params

@PakjeBecel
Copy link
Contributor Author

You are right, I did forget about those pages as well, I will add them too. Thank you for mentioning!

@PakjeBecel PakjeBecel marked this pull request as draft April 24, 2025 09:17
Updated both desction with the new components v2 behaviour.
Updated the section with the asterisk being moved from the "flags" description to the "flags" field. Removed one of the asterisks since only one was present inside the table.
@PakjeBecel PakjeBecel marked this pull request as ready for review April 24, 2025 09:46
@PakjeBecel PakjeBecel requested a review from Lulalaby April 24, 2025 10:36
@PakjeBecel
Copy link
Contributor Author

PakjeBecel commented Apr 24, 2025

I might need to check the Interactions: Receiving and Responding docs too, since the behaviour of components v2 isn't mentioned at all there, while it is mentioned on the Components: Using Message Components that components v2 can be used as response to an interaction too.

Edit: saw that the "responding to an interaction" text links to the "Create Followup Message" section, where it already stated it functions the same as "Execute Webhook", so there is no need to add anything related to components v2 in my opinion.

@PakjeBecel PakjeBecel changed the title Updated "Message" resource page and "Using Message Components" page with the new components v2 behaviour Updated "Message" and "Webhook" resource page and "Using Message Components" page with the new components v2 behaviour Apr 24, 2025
@colinloretz colinloretz requested review from anthonydiscord and removed request for Lulalaby May 8, 2025 20:22
@colinloretz colinloretz added the message components click click click click click click click label May 8, 2025
@anthonydiscord
Copy link
Contributor

Thank you for taking the time to document these. I knew there would be a few spots we'd miss when writing this so I appreciate you giving it a thorough pass! Looks like something is up with the tables if you don't mind running npm run fix:tables once more and then this looks good to me

@anthonydiscord anthonydiscord merged commit a7dfd25 into discord:main May 12, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
message components click click click click click click click
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants