Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

allow for matching on any nickname mention #210

Closed
wants to merge 1 commit into from

Conversation

crunchywelch
Copy link

sometimes we like to alert in channel with just the nick, without a proper @nickname mention, this allows for a broader match for any mention of the nick such as nickname: message this patch adds a broad_mention message event type which catches those instances

@anonrig
Copy link
Contributor

anonrig commented May 7, 2016

Hey @crunchywelch

Thank you for your pull request. Honestly, it's really great the ability to send a broad_mention event upon a message that includes a username, but I am not really quite sure if this functionality should be added to the library itself or should be added to an example project itself.

What about we wait for other responses/thoughts in this manner? :)

@benbrown
Copy link
Contributor

Let's use the version of this functionality in this PR:

#189

@benbrown benbrown closed this May 23, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants