-
-
Notifications
You must be signed in to change notification settings - Fork 187
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
Clarification of MIGRATION.md #278
Comments
It will for as long as Slack supports RTM apps in any form or shape. I have half a dozen bots using it with hundreds of paying customers, and haven't had the energy to migrate them all, yet.
This is no longer necessary because events are sent by Slack to your bot. So there's no "connection" anymore. If you don't want 1-click "add to slack" install type of thing that is available to the public, you can probably just remove that functionality from your own bot. It would definitely be useful to have a way to do this more easily, too, PRs welcome.
If you set the right permissions (I am not sure exactly what permissions you need to listen to everything) it will just work.
There's no no-pain solution here, but I tried to minimize the work required by writing https://github.com/slack-ruby/slack-ruby-bot-server-events-app-mentions which is sort of very close to the functionality of the slack-ruby-bot alone. It does reply to mentions. See https://code.dblock.org/2020/11/30/migrating-classic-slack-ruby-bots-to-granular-permissions.html for lots of detail. I'll close this, but feel free to add more questions! |
Got answer from Slack support.
Basically flow will to migrate from really legacy bots:
|
@mr-exz I am confused why you'd want to do this and not create an app with granular permissions in the first place? |
@dblock that is good question, when i create granular directly i got error about bad token not_allowed_token_type |
Oh I see. You can't start an RTM client unless you have a legacy RTM token, even with granular permissions. Still, feels like you should just not be doing RTM at all. |
it require many changes in application, so i need more time, thx for clarification will create separate task for myself to migrate from RTM |
Answer from Slack Support We don't have plans to deprecate Classic Apps at this stage (no time frame) so you can still use them for now. However it is possible that this could change (you would receive a warning to then update before this happened though). We recommend moving to Socket Mode if possible: https://api.slack.com/apis/connections/socket You can see a list of events that are supported in Socket Mode (Events API) vs RTM below. Anything not listed as Events API but shows as RTM is not possible with Socket Mode. |
Oh socket mode is yet another thing! I opened slack-ruby/slack-ruby-bot-server#141 for that if anyone wants to contribute. |
As i understood from MIGRATION.md we have to migration application to OAuth but from Slack documentation they still allow create classic apps but they ask migrate app (bot) to granular permissions and thats it. In the bottom of that page they allow create classic app https://api.slack.com/legacy/oauth#flow.
https://api.slack.com/apps?new_classic_app=1
So recently i created new test classic app (bot) got api token
xoxb-
and migrated to granular permissions so all warnings disappear.So my questions:
@mybot help
The text was updated successfully, but these errors were encountered: