Skip to content
This repository was archived by the owner on Jul 16, 2022. It is now read-only.

Fixed Invite.inviter (extensions) as was empty#79

Merged
HilbertGilbertson merged 2 commits intoGAwesomeBot:developmentfrom
HilbertGilbertson:fixinviteinviter
Feb 14, 2017
Merged

Fixed Invite.inviter (extensions) as was empty#79
HilbertGilbertson merged 2 commits intoGAwesomeBot:developmentfrom
HilbertGilbertson:fixinviteinviter

Conversation

@HilbertGilbertson
Copy link
Copy Markdown
Collaborator

@HilbertGilbertson HilbertGilbertson commented Feb 12, 2017

As has been the problem with numerous other properties, Inviter was a getter. This meant the invite creator's user object wasn't passing with the Invite object and wasn't retrievable (an extension could not even see which user created the invite).

To reproduce: guild.getInvites(cb => {logMsg(JSON.stringify(cb))});

Resolved by always passing the User object via .inviter.

As has been the problem with numerous other properties, Inviter was a getter.
This meant the invite creator's user object wasn't passing with the Invite
object and wasn't retrievable (an extension could not even see which user
created the invite).

To reproduce: `guild.getInvites(cb => {logMsg(JSON.stringify(cb.inviter))});`

Resolved by always passing the User object via .inviter.
@HilbertGilbertson HilbertGilbertson added reason: Bug Issues that are caused by a deviancy in GAB's behavior target: Extensions Issues that target the Extension handler or API labels Feb 12, 2017
@vladfrangu vladfrangu self-requested a review February 14, 2017 20:54
Copy link
Copy Markdown
Member

@vladfrangu vladfrangu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merge-ready

@HilbertGilbertson HilbertGilbertson merged commit a5cddf9 into GAwesomeBot:development Feb 14, 2017
@HilbertGilbertson HilbertGilbertson deleted the fixinviteinviter branch February 14, 2017 21:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

reason: Bug Issues that are caused by a deviancy in GAB's behavior target: Extensions Issues that target the Extension handler or API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants