Skip to content

refactor: remove built-in slack backend#1581

Merged
sijis merged 10 commits intoerrbotio:masterfrom
sijis:refactor/slack
Jul 21, 2022
Merged

refactor: remove built-in slack backend#1581
sijis merged 10 commits intoerrbotio:masterfrom
sijis:refactor/slack

Conversation

@sijis
Copy link
Copy Markdown
Contributor

@sijis sijis commented Jun 20, 2022

Remove built-in (deprecated) slack backends in favor of slackv3.

@sijis sijis force-pushed the refactor/slack branch 3 times, most recently from dd3fcd8 to f8355cc Compare June 21, 2022 00:09
@sijis sijis requested a review from nzlosh June 22, 2022 03:42
@sijis sijis marked this pull request as ready for review June 22, 2022 03:42
@sijis
Copy link
Copy Markdown
Contributor Author

sijis commented Jun 22, 2022

@nzlosh Could you look over slack documentation and make sure it looks right?

Copy link
Copy Markdown
Contributor

@nzlosh nzlosh left a comment

Choose a reason for hiding this comment

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

This looks good to me.

@sijis sijis merged commit b1a6408 into errbotio:master Jul 21, 2022
@sijis sijis deleted the refactor/slack branch July 21, 2022 03:04
sijis added a commit to sijis/errbot that referenced this pull request Jan 1, 2024
* refactor: remove slack backend

* refactor: remove slack-rtm backend

* test: remove slack tests

* docs: remove slack and use slackv3

* chore: add slackv3 build to Dockerfile

* feat: enable backend plugins directory on --init

* chore: add slackv3 repo to backend image

* refactor: simplify the final image

* docs: fix slack refs in README

* docs: add info to CHANGES
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants