-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
feat(Webhook): add '(edit|delete)Message' methods #5223
Conversation
2f86981
to
469e9e4
Compare
df34e7d
to
899abe3
Compare
899abe3
to
c917879
Compare
This PR probably needs a rebase, I suspect that the duplicated type is from this PR and #4876. |
typings(WebhookFields): add '(edit|delete)Message' methods chore(typings): merge upstream changes refactor(typings): use WebhookRawMessageResponse feat(typings): add stricter overloads for 'WebhookClient' fix(typings): include 'content' in 'WebhookMessageOptions' docs(WebhookMessageOptions): add 'content' prop docs(Webhook): create WebhookEditMessageOptions typedef
…rd.js into feat/webhook-methods
Added the option to allow Code used to test this change (which has also been added within the test code in the PR description)
|
Attachments can be uploaded when editing a webhook message (discord/discord-api-docs#2615 (comment), discord/discord-api-docs#2615 (comment)) |
Added the option to edit the message with Code used to test this change (which has also been added within the test code in the PR description)
|
Adds support for the Edit Webhook Message and Delete Webhook Message endpoints.
After #4876 is merged, the(See df89d04)object
that represents the raw message data in the typings can be replaced to be more accurate with eitherWebhookRawMessageResponse
(or eventuallyAPIMessage
over ondiscord-api-types
).Here's a snippet of how I tested these changes
Status
Semantic versioning classification: