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

Fix direct mentions #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

EmmaSimon
Copy link

Mentions were ignored previously, because the bot ID was never set on
any of the routes. This sets the ID when the bot is run.

Mentions were ignored previously, because the bot ID was never set on
any of the routes. This sets the ID when the bot is run.
@wolftrouble
Copy link

@chris-skud Any hope of this being merged? Easy enough to patch on our end but this is such a simple fix I'm surprised it's not been merged.

@lucymhdavies
Copy link

lucymhdavies commented Dec 15, 2016

@EmmaSimon does this still work? I've been trying your fork with the simple.go example, but no luck.

Maybe I'm doing something wrong.

@wolftrouble
Copy link

@lucymhdavies The PR worked for me on head. At this point though someone should probably fork this project and maintain it separately, it's pretty obvious @BeepBoopHQ isn't super interested in maintaining it anymore.

@lucymhdavies
Copy link

lucymhdavies commented Dec 16, 2016

Ah, my bad. I was still importing the BeepBoopHQ version.
Our version works a treat :)

We're going to be using it a lot in our company soon, so our fork is:
https://github.com/adampointer/go-slackbot

@CK-Ward
Copy link

CK-Ward commented Oct 17, 2017

Can we please merge this? :)

flw-cn pushed a commit to flw-cn/go-slackbot that referenced this pull request Jun 25, 2018
liamsorsby pushed a commit to liamsorsby/go-slackbot that referenced this pull request Jul 7, 2022
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.

4 participants