Skip to content

[Slack] Use socket mode instead of RTM API#1744

Open
DavyJohnes wants to merge 4 commits into
42wim:masterfrom
DavyJohnes:socket-mode
Open

[Slack] Use socket mode instead of RTM API#1744
DavyJohnes wants to merge 4 commits into
42wim:masterfrom
DavyJohnes:socket-mode

Conversation

@DavyJohnes
Copy link
Copy Markdown
Contributor

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.

@42wim 42wim added the slack label Mar 11, 2022
@42wim
Copy link
Copy Markdown
Owner

42wim commented Mar 11, 2022

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

@DavyJohnes
Copy link
Copy Markdown
Contributor Author

DavyJohnes commented Mar 15, 2022

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?

@42wim
Copy link
Copy Markdown
Owner

42wim commented Mar 15, 2022

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

@qlty-cloud-legacy
Copy link
Copy Markdown

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

View more on Code Climate.

@GoliathLabs
Copy link
Copy Markdown
Contributor

Any updates on this?

@unode
Copy link
Copy Markdown

unode commented Sep 23, 2024

Related #2161

@BolajiAyodeji
Copy link
Copy Markdown

Hi! Any updates on this PR? Seems to have stalled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants