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.

Sign up for GitHub

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

Jump to bottom

[Feature] Allow users to inject context #2

Open
curtisallen opened this issue Feb 21, 2016 · 0 comments
Open

[Feature] Allow users to inject context #2

curtisallen opened this issue Feb 21, 2016 · 0 comments

Comments

@curtisallen
Copy link

It would be nice if users of this package were allowed to inject context. I'm running into a issue where I'd like some of my context values in handlers.

Maybe we can add a context arg to New

slackbot.New(slackToken string, ctx context.Context)

Then in Run we check for a user provided context if not we create a fresh context.
flw-cn pushed a commit to flw-cn/go-slackbot that referenced this issue Jun 25, 2018
@lusis
cleaning up all the regexp.MustCompile stuff (BeepBoopHQ#2)
e2bf591
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@curtisallen