-
-
Notifications
You must be signed in to change notification settings - Fork 727
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
Add support for role icons #1823
Add support for role icons #1823
Conversation
Looks pretty good to me. @MinnDevelopment? |
The docs PR is only a datamine PR for an experiment, so I would be against merging it at this time. See discord/discord-api-docs#3815 |
that's one of the reasons this PR is a draft. I haven't tested it |
Co-authored-by: Austin Keener <keeneraustin@yahoo.com>
Okay, going to mark as freezer until the feature is no longer experimental. |
seems like the PR has been merged. I'll still wait until the feature is fully released to test it out properly |
src/main/java/net/dv8tion/jda/internal/managers/RoleManagerImpl.java
Outdated
Show resolved
Hide resolved
…l.java Co-authored-by: Florian Spieß <business@minnced.club>
it'd be great if someone with access to role icons (owning a server with boost tier 2+) could test this PR because I don't have one. (I think the PR should be fine though) |
When will this be available? I'm waiting for it |
PR needs resolution of conversation between @caneleex and @MinnDevelopment before PR can be merged |
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.
Only a small clarification in docs and then its good to go
Co-authored-by: Florian Spieß <business@minnced.club>
Pull Request Etiquette
Changes
Closes Issue: NaN
Description
this PR adds support for role icons. see discord/discord-api-docs#3847