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

Use socket mode events #10

Merged
merged 6 commits into from
Oct 25, 2022
Merged

Conversation

surdaft
Copy link
Collaborator

@surdaft surdaft commented Oct 25, 2022

Slack have deprecated the RTM API and recommend using the socket mode API instead.

https://api.slack.com/rtm
https://api.slack.com/apis/connections/socket

New Slack apps may not use any Real Time Messaging API method
For most applications, Socket Mode is a better way to communicate with Slack.

I have altered the documentation and updated the code to allow the use of this new API. In addition I added a docker build action to allow the use of the memo system through a precompiled docker image.

This is a backwards breaking change due to the requirement of an app token to create the socket connection.


Any feedback on this is very much recommended, I intend to add some QOL updates as time progresses, as I enjoy using this convenient memo system.

@CLAassistant
Copy link

CLAassistant commented Oct 25, 2022

CLA assistant check
All committers have signed the CLA.

@surdaft
Copy link
Collaborator Author

surdaft commented Oct 25, 2022

The dockerfile is only vaguely tested due to working from a private corporation network, and so the SSL is failing to validate. External compilation works fine though and so in theory it should be all good in there. Would be worth a test though.

@davemoore- davemoore- merged commit ba10c1d into grafana:master Oct 25, 2022
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

3 participants