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(GuildMemberManager): nick endpoint #7151
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.
Thank you for catching this!
Can't we just change that to |
@kyranet's suggestion seems the most future-proof, so I went with that |
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.
Are you sure Kyra's suggestion works/has it been tested? If it works, LGTM
I tested it and it worked |
Please describe the changes this PR makes and why it should be merged:
#7104 introduced a regression to
Guild#me#setNickname
sincecommunication_disabled_until
is always added to_data
, even ifcommunicationDisabledUntil
isundefined
. Due to thisObject.keys(_data).length
will be 2 and the check in line 272/274 won't add@me
to the endpoint.Status and versioning classification: