Skip to content

Support socket mode#42

Open
dummyx wants to merge 4 commits into
docmarionum1:masterfrom
dummyx:master
Open

Support socket mode#42
dummyx wants to merge 4 commits into
docmarionum1:masterfrom
dummyx:master

Conversation

@dummyx
Copy link
Copy Markdown

@dummyx dummyx commented Aug 16, 2022

Add support for socket mode in newer versions of bolt, which eliminates the need of a running http server.

It can be enabled through the command line switch "-s" or "--socket".

@docmarionum1
Copy link
Copy Markdown
Owner

Change looks good. Can you update the documentation (README.md) with how to use the new functionality?

@dummyx
Copy link
Copy Markdown
Author

dummyx commented Aug 17, 2022

Change looks good. Can you update the documentation (README.md) with how to use the new functionality?

OK, I'll work on it.

@dummyx
Copy link
Copy Markdown
Author

dummyx commented Aug 22, 2022

Change looks good. Can you update the documentation (README.md) with how to use the new functionality?

@docmarionum1 Added some minimal documentation for the socket mode.

Comment thread README.md

## Socket mode

To run the bot in socket mode, go to the app's "Socket Mode" page and enable it. Then add the app level token you got to environment variables.
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
To run the bot in socket mode, go to the app's "Socket Mode" page and enable it. Then add the app level token you got to environment variables.
To run the bot in socket mode, go to the app's "Socket Mode" page and enable it. Then add the app level token you got to environment variables under the name `SLACK_APP_TOKEN`.

Copy link
Copy Markdown
Owner

@docmarionum1 docmarionum1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made a small suggested change for clarity in the readme. Otherwise looks good. Thanks!

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.

2 participants