-
Notifications
You must be signed in to change notification settings - Fork 2k
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
slackbot example update to reflect package and api changes #879
Conversation
hmm, sorry... can someone remind me of the node linter I should use? |
Ran the linter on the sample with `npm run lin -- --fix`.
Thanks! I ran the linter and pushed ( |
I'm not sure if we ever had one. Tim -- you have also created some slackbot examples... do you know if we have a test token? If not, I can generate one. @tswast |
I created https://clouddpes.slack.com/ a while back to test Slack samples. I've still got a few apps at https://api.slack.com/apps I'm not sure how to get a Slack API token anymore. I think they've moved to more of an OAuth2-based flow as their recommended auth mechanism. |
It looks like it's still possible to get this kind of 'bot' token, IIUC: https://api.slack.com/docs/token-types#bot |
ping @amygdala Can you have a look at the merge conflict. Thanks! |
ping @amygdala |
ooops, sorry. This had popped off my stack. Will do! |
…samples into sbotfix
(ughh, I need to figure out how to lint with node. Clearly more is needed in terms of setup than I have installed. I'll take a look on the weekend) |
I'm rethinking: I'm having trouble finding the bandwidth to deal with this right now, and it's kind of an orphaned example anyway-- how about I just move it into one of my own repos and relink the blog post? |
Getting back to this after the holidays. I'm closing this PR, and will just move the code into one of my own repos. |
No description provided.