-
-
Notifications
You must be signed in to change notification settings - Fork 264
Support Components V2 #1294
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
Support Components V2 #1294
Conversation
|
Example. |
- Missing what do with message methods for components
|
I think we should rename "ActionComponent" to "InteractiveComponent" to match discord's documentation |
|
I don't think that |
sounds good currently already make break changes based in the start of the components here xd |
hmmm i mean its true but just keep MessageComponent also feels strange for later how handle in rows... |
- Reorder hierarchy with IAccessoryComponent, ICanBeUsedInContainerComponent and ICanBeUsedInSectionComponent - Add component id support - Add missing load state and proxy url for UnfurledMediaItem - Fix many javadocs
0f1a001 to
e81fd17
Compare
Azn9
left a comment
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.
Looks good for me for a release!
|
maybe can mention this in the components/message class https://github.com/Lulalaby/discord-api-docs/compare/main...comp_v2?expand=1#diff-5ce2876134b072c51fe6cbd0ab97e72556e2a37507fb90f9bb5a474debb5d976R680 |
Description: Add support for Components V2
Depends on: Discord4J/discord-json#198
Notes: