-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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(Widget): wrapper for widget.json #5619
Conversation
Co-authored-by: monbrey <[email protected]>
It's a mess in the logs but I've added in client an Promise.reject in Client#fetchWidget + clean up the if condition in the Widget class |
I have add a new function, exactly the same as the Client#fetchWidget but in the Widget to update the widget |
CamelCase Co-authored-by: BannerBomb <[email protected]>
CamelCase Co-authored-by: BannerBomb <[email protected]>
I have done a test but that change anything I guess ? I've just link my local to my github project (3ead9e6) |
Co-authored-by: Antonio Román <[email protected]>
…e changed, moved WidgetMember#_patch to constructor
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.
Otherwise LGTM
Co-authored-by: SpaceEEC <[email protected]>
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.
Just some formatting things for consistency, otherwise LGTM
Something I didn't see was the property fetchWidget in Guild (Guild#fetchWidget). |
Before push it, do we need add a Guild#fetchWidget like I said upper ? |
Please describe the changes this PR makes and why it should be merged:
Okay, I'm remaking a new pr because my last was really terrible + i've deleted the original (#5030)
My goal is to add Widget in discord.js
Widget is a part of Discord and it's not fully featured
Status and versioning classification:
Thanks to @ckohen @advaith1 @SpaceEEC @almostSouji @vladfrangu @NotSugden @kyranet @Mesteery for helping me in the last pull request