Skip to content
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

track "/me" IRC actions #147

Merged
merged 1 commit into from Jun 23, 2015
Merged

Conversation

ktdreyer
Copy link
Contributor

Twisted tracks normal IRC messages using the privmsg() method and "/me" IRC messages using the action() method.

This pull request aliases the action method to the privmsg method so that Helga will process both types of messages. This change allows plugins to trigger on content in "/me" IRC messages.

Twisted tracks normal IRC messages using the privmsg() method and "/me"
IRC messages using the action() method.

Alias the action method to the privmsg method so that Helga will process
both types of messages. This change allows plugins to trigger on content
in "/me" IRC messages.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling c12f673 on ktdreyer:track-irc-actions into 1f53700 on shaunduncan:master.

@shaunduncan
Copy link
Owner

Thank you for this! I think this will help me out quite a bit for #128. If it's alright, I'd like to experiment a bit with this to see some of the behavior with respect to the channel logger. I think it'll probably be fine, but I just wanted to make sure.

@ktdreyer
Copy link
Contributor Author

Hi Shaun, @zmc and I were discussing this again today. Does it look ok to merge?

@shaunduncan
Copy link
Owner

Oh my gosh, I'm so sorry this went stale for a while. It all looks good to me. I'll see about getting a new release out tomorrow.

shaunduncan added a commit that referenced this pull request Jun 23, 2015
@shaunduncan shaunduncan merged commit 9864240 into shaunduncan:master Jun 23, 2015
@shaunduncan
Copy link
Owner

Just released helga 1.7.2 to include this change

@ktdreyer ktdreyer deleted the track-irc-actions branch February 15, 2016 17:06
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.

None yet

3 participants