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.

Sign up for GitHub

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

Jump to bottom

Remove outdated logic and tests related to Slack API update #5

Open
nickiemc opened this issue Oct 18, 2022 · 0 comments
Open

Remove outdated logic and tests related to Slack API update #5

nickiemc opened this issue Oct 18, 2022 · 0 comments

Comments

@nickiemc
Copy link

As noted in ein-slackbot issue #180, the API method used to connect to Slack, rtm.start, was deprecated and its response changed on September 27, 2022.

corgi-lita-slack was updated in PR #4 to instead use rtm.connect to connect to Slack. PR #4 included the bare minimum changes. As part of this issue, remaining outdated logic and tests related to the change in the response returned from the Slack API should be appropriately cleaned up.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nickiemc