matterbridge icon indicating copy to clipboard operation
matterbridge copied to clipboard
verified publisher icon 42wim

[Slack] Use socket mode instead of RTM API

Open DavyJohnes opened this issue 3 years ago • 7 comments

RTM API is petty old api interface and enforces to use old-style Slack Apps (also known as "Classic" apps) which doesn't allow to use some new features (like org-wide app installations). Slack itself recommends to use Socket Mode (https://api.slack.com/rtm) for most applications.

If this approach is acceptable I can provide documentation how to install and configure Slack App to use it with socket mode.

DavyJohnes avatar Feb 18 '22 16:02 DavyJohnes

Thank you for this PR 👍 and sorry for the delay in responding, could you fix the golangci-lint issues? Trying to test this PR this weekend and give feedback

42wim avatar Mar 11 '22 23:03 42wim

Thank you for this PR 👍 and sorry for the delay in responding, could you fix the golangci-lint issues? Trying to test this PR this weekend and give feedback

Sure, could you please restart workflow in order to get fresh error after rebase?

DavyJohnes avatar Mar 15 '22 10:03 DavyJohnes

Thanks, you can ignore the exhaustive linter, going to remove it, but if possible fix the other issues

42wim avatar Mar 15 '22 20:03 42wim

Code Climate has analyzed commit 03d78007 and detected 0 issues on this pull request.

View more on Code Climate.

qlty-cloud-legacy[bot] avatar Mar 25 '22 21:03 qlty-cloud-legacy[bot]

Any updates on this?

GoliathLabs avatar Jul 16 '22 11:07 GoliathLabs