discord-api-docs icon indicating copy to clipboard operation
discord-api-docs copied to clipboard

Update Edit Message to reflect that not every parameter is nullable

Open onerandomusername opened this issue 3 months ago • 5 comments

These parameters are not all nullable.

onerandomusername avatar Sep 22 '25 16:09 onerandomusername

This is a breaking change, no? What happened?

Jiralite avatar Sep 22 '25 19:09 Jiralite

Is this PR supposed to remove just the nullable callout from the admonition? Because making PATCH not have optional fields is a super breaking change, and I hope that didn't happen

vladfrangu avatar Sep 22 '25 19:09 vladfrangu

All patch params are optional and always will be optional.

advaith1 avatar Sep 22 '25 19:09 advaith1

Apologies, deleted a bit too much. Edited this to be in alignment with what seems to be the actual API behaviour, and with what DV8 committed earlier.

onerandomusername avatar Sep 23 '25 05:09 onerandomusername

I was playing around with this and poll, attachments, embeds, and flags can all be null (as long as their is some valid content for the message). Likely some of the others can be as well? Am I missing something or are there any that can't be null? I do think it makes sense to add * for all nullable options even though we call it out above

anthonydiscord avatar Oct 02 '25 21:10 anthonydiscord