Skip to content
This repository was archived by the owner on Sep 22, 2025. It is now read-only.

docs: Add brief migration guide#260

Merged
dblock merged 4 commits into
slack-ruby:masterfrom
wasabigeek:add-migration-docs
May 19, 2020
Merged

docs: Add brief migration guide#260
dblock merged 4 commits into
slack-ruby:masterfrom
wasabigeek:add-migration-docs

Conversation

@wasabigeek
Copy link
Copy Markdown
Contributor

Fixes #252.

I may not have the opportunity to verify this flow for some time, if anyone has tried it, do let me know if it works / there are any issues!

@wasabigeek wasabigeek force-pushed the add-migration-docs branch from a61f8c4 to 9b37a82 Compare May 8, 2020 16:03
@wasabigeek wasabigeek force-pushed the add-migration-docs branch from 9b37a82 to 73ceea5 Compare May 8, 2020 16:04
Copy link
Copy Markdown
Collaborator

@dblock dblock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good start!

Comment thread MIGRATION.md Outdated
Comment thread MIGRATION.md Outdated
Comment thread MIGRATION.md Outdated

## Migrating to slack-ruby-bot-server
Overview:
- Create a new Slack App as [advised by Slack](https://api.slack.com/legacy/custom-integrations/legacy-tokens#migrating-from-legacy).
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is confusing. I think we should tell users to go through the tutorial/setup of slack-ruby-bot-server as in that doc, then copy command classes, then do the workflow that you suggest. And explain what happens when you do that and the fact that other teams can install the bot too then, which may not be what you want (and needs to be prevented).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am actually a little confused by Slack itself. If I understood correctly, slack-ruby-bot-server requires the latest Slack Apps, but there are also legacy tokens AND class Slack Apps, both with subtly different migration strategies on the Slack end. Did I get that right, and should I gloss over this?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I say gloss over this.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok! Any more comments?

@wasabigeek wasabigeek marked this pull request as ready for review May 18, 2020 15:34
@dblock dblock merged commit 60c8ad0 into slack-ruby:master May 19, 2020
@dblock
Copy link
Copy Markdown
Collaborator

dblock commented May 19, 2020

I merged this, thank you.

@wasabigeek wasabigeek deleted the add-migration-docs branch May 20, 2020 02:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migration to non-legacy bots

2 participants