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
fix(MessageComponentInteraction): component getter is not null anymore #6835
Conversation
Update typings and also please follow the PR template |
no problem. will do [Update 16:56]: Added relevant changes to typings, PR adheres to template now. |
Updated typings/index.d.ts
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.
updated relevant files.
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.
changes applied to MessageCompInteraction
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.
changes according to ESLint.
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.
Maybe change title to fix(MessageComponentInteraction): component getter is not nullable anymore
Changed title to the standard of template. |
Couldn't this still be null in a partial Message? |
Hi, from another quick view around the codebase, I believe MessageComponent itself doesn't set any null data irrespective of any partial or normal message event updates in it. Check out file: The updates mentioning the changed lines of issue #6481 include customId which also cannot be null /whole component cannot default to null. |
Please describe the changes this PR makes and why it should be merged:
This refers issue #6481 - The component getter of a MessageComponentInteraction cannot be null.
Removed null coal. from both TS typings/main getter of the component.
(following init commit of d6e6244)
Status and versioning classification: