Skip to content
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

Document bulk ban endpoint #6720

Merged
merged 4 commits into from
Mar 30, 2024
Merged

Conversation

MinnDevelopment
Copy link
Contributor

@MinnDevelopment MinnDevelopment commented Mar 17, 2024

This endpoint has some interesting pitfalls when a user is not banned.

  • If a user was already banned, they are in the "failed_users" array
  • If you don't have permissions to ban a user, they also appear in failed users
  • If all users "failed" you get an error response instead
  • The endpoint filters out the self user (does not show up in either array)

docs/resources/Guild.md Outdated Show resolved Hide resolved
Co-authored-by: advaith <advaithj1@gmail.com>
Co-authored-by: Adrian Paschkowski <adrian.paschkowski@ruhr-uni-bochum.de>
@NeloBlivion
Copy link

NeloBlivion commented Apr 22, 2024

Is this endpoint working as intended? Used it to ban 150 users, one was already banned so it comes down to 149 successful, 1 failed. However, i still got 500000 Failed to ban users instead of the two arrays.
In another server where all bans were successful, it worked as intended.

Nihlus added a commit to Remora/Remora.Discord that referenced this pull request Apr 29, 2024
shaydewael pushed a commit to Jupith/discord-api-docs that referenced this pull request May 14, 2024
* Document bulk ban endpoint

* Add manage guild permission requirement

Co-authored-by: advaith <advaithj1@gmail.com>

* Update docs/resources/Guild.md

Co-authored-by: Adrian Paschkowski <adrian.paschkowski@ruhr-uni-bochum.de>

* Fix nullability

---------

Co-authored-by: advaith <advaithj1@gmail.com>
Co-authored-by: Adrian Paschkowski <adrian.paschkowski@ruhr-uni-bochum.de>
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.

None yet

5 participants