Skip to content

Include message object with emit#57

Closed
alexwhitman wants to merge 10 commits intomartynsmith:masterfrom
alexwhitman:emit-messages
Closed

Include message object with emit#57
alexwhitman wants to merge 10 commits intomartynsmith:masterfrom
alexwhitman:emit-messages

Conversation

@alexwhitman
Copy link
Copy Markdown
Contributor

I need access to the message object so I've added it to the emit calls where it makes sense (I ignored whois/chanlist etc where the payload is built up with other messages)

@martynsmith
Copy link
Copy Markdown
Owner

This looks fine to me, if you could update the documentation too then I'd be happy to pull this.

Otherwise I'll try get around to it :-)

@alexwhitman
Copy link
Copy Markdown
Contributor Author

Damnit, looks like I messed something up when re-basing. I'll close this pull request and open a new one.

Half-Shot referenced this pull request in matrix-org/node-irc Feb 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants