-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
feat: Use organization scoped adapters like @hubot-friends/hubot-slack #1655
Conversation
BREAKING CHANGE: This change now requires you to include hubot- when specifying hubot adapters.
🎉 This PR is included in version 7.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Hi, I see that you reference "@hubot-friends/hubot-slack". What is that? (I'm looking for a new Slack adapter that uses Bolt instead of the RTM API) |
Found it! Awesome!! https://www.npmjs.com/package/@hubot-friends/hubot-slack |
Yes, that's the one I'm referring to. Please note that it's a fork of the old hubot-slack adapter and EXTENSIVELY modified to utilize @slack/web-api and @slack/socket-mode, replacing Slacks RTM client from @slack/client. It doesn't use Bolt. I would love your critical feedback if you use it though. |
At Wehkamp we have a jump-start project for Hubot / Slack chat bots. We have a public base here: https://github.com/wehkamp/bot-zero With some internal bots we try to add dialog features to the bot, for which we might need bolt (but I'm not sure if I fully understand the difference between socket mode and bolt, as Slack muddies the communication a bit). The main things we don't want to loose are hubot-grafana and hubot-pager-me, so we're really happy that you came up with something that works with a modern Slack app. Super kudos! |
@joeyguerra looks like we might need to add support to hubot-grafana for the special |
@joeyguerra can you check hubot-friends/hubot-slack#21 for better error handling? I think if we have that in the package we get a better handle on what's going wrong if the Slack API does not want to eat our request. |
@KeesCBakker yes. Will collaborate more over there. |
BREAKING CHANGE: This change now requires you to include hubot- when specifying hubot adapters.
fixes #1654