monzo / response

Monzo's real-time incident response and reporting tool ⚡️
MIT License
1.52k stars 165 forks source link

Remove deprecated Slack channel API calls. Use conversations API. #220

Closed sjdweb closed 4 years ago

sjdweb commented 4 years ago

Fixes #218

Pay particular attention to conversations.join -- the API is subtly different; Channel ID is required and not channel name.

Edit: there was also a change around conversations.invite. I updated this (send a list of users vs. single user), however you do get an error:

ERROR - action_handlers - Error calling Slack API endpoint 'conversations.invite': cant_invite_self

It seems this inviting of the bot user is unnecessary, as the bot joins the channel anyway during creation.