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

Published 20 hours ago • verified publisher icon discord

Document ban `delete_message_seconds` and remove `reason`

Open advaith1 opened this issue 1 year ago • 5 comments

Updated banning docs:

  • Added delete_message_seconds
  • Marked delete_message_days as deprecated
  • Removed reason, which was removed in v10

advaith1 avatar Jul 20 '22 23:07 advaith1

reason is replaced with the X-Auditlog-Reason Header right?

Lulalaby avatar Jul 20 '22 23:07 Lulalaby

reason is replaced with the X-Auditlog-Reason Header right?

As the v10 changelog says, yes

ooliver1 avatar Jul 21 '22 00:07 ooliver1

Does the new field accept an arbitrary number of seconds or is it a predefined list?

splatterxl avatar Jul 21 '22 00:07 splatterxl

any number in the range

advaith1 avatar Jul 21 '22 02:07 advaith1

Testing this out, it seems to be overreaching. With 4 messages spaced out about 2 minutes each, a delete_message_seconds value of 60 deletes all 4. Tested in both v9 & v10, same results in both. Is this not fully ready yet, or is there some invisible rounding happening behind the scenes?

DonovanDMC avatar Jul 22 '22 08:07 DonovanDMC