Skip to content
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

Add deprecation notice to Slack and SlackRTM backends #1526

Merged
merged 3 commits into from
Dec 11, 2021

Conversation

sijis
Copy link
Contributor

@sijis sijis commented Jul 22, 2021

Adding deprecation notices to these backends and suggesting the SlackV3 one.

In the logs, the message looks like

00:03:28 INFO     errbot                    webhooks:  Flag to bind /github to notification
00:03:28 INFO     errbot                    webhooks:  Flag to bind /test to test
00:03:28 WARNING  errbot.backends.slack     This backend is deprecated and will be removed in a future release. No future updates, bug fixes or enhancements will be included. We strongly advise migrating to SlackV3, which is available at https://github.com/errbotio/err-backend-slackv3
00:03:28 INFO     errbot.backends.slack     Verifying authentication token
00:03:28 INFO     errbot.backends.slack     Connecting to Slack real-time-messaging API
00:03:29 INFO     errbot.backends.slack     Connected
```

@sijis sijis force-pushed the chore/deprecate_builtin_slack branch from 6d806bf to 2c987a3 Compare December 3, 2021 06:52
@sijis sijis marked this pull request as ready for review December 3, 2021 06:58
@sijis sijis merged commit db9e0bc into errbotio:master Dec 11, 2021
@sijis sijis deleted the chore/deprecate_builtin_slack branch December 11, 2021 22:58
sijis added a commit that referenced this pull request Jun 11, 2022
* fix: add deprecation notice to Slack backend

* fix: add deprecation notice to SlackRTM backend

* docs: Add backend deprecation 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.

None yet

1 participant