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: GuildBanManager #5276
feat: GuildBanManager #5276
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
jsdoc also needs correcting
Co-authored-by: Sugden <28943913+NotSugden@users.noreply.github.com>
Co-authored-by: Sugden <28943913+NotSugden@users.noreply.github.com>
Co-authored-by: Sugden <28943913+NotSugden@users.noreply.github.com>
Co-authored-by: Jan <66554238+vaporox@users.noreply.github.com>
Co-authored-by: Jan <66554238+vaporox@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
jsdoc strings for the events still need updating
Co-authored-by: Jan <66554238+vaporox@users.noreply.github.com>
Co-authored-by: Jan <66554238+vaporox@users.noreply.github.com>
…nto banmanager
This reverts commit 598ba38.
… discordjs-master
Co-authored-by: Sugden <28943913+NotSugden@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems good to me
Co-authored-by: Shubham Parihar <shubhamparihar391@gmail.com>
|
Co-authored-by: Shubham Parihar <shubhamparihar391@gmail.com>
Please describe the changes this PR makes and why it should be merged:
This PR adds
GuildBanManager
to guilds, it replacesguild#fetchBans
/guild#fetchBan
and adds a ban cache (which is most of the time partial). I decided to add this because actions like fetching guild ban count or checking if a member is banned required sending a request to Discord every single time (or manually handling ban/unban events lol)I've been using this with my bots for probably 4 months now and didn't have any issues (afaik)
(a lot of comments/docs are copied from other classes lol)
Status and versioning classification:
Please be gentle, this is my first time