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

Enhance: Add support to pull slack bot token #451

Merged
merged 1 commit into from
Feb 20, 2025

Conversation

StrongMonkey
Copy link
Contributor

Add support to use slack bot token to send message a bot. This is done by adding a new tool into slack bundle that consumes a separate credential that requests bot token.

@@ -12,7 +12,7 @@ Tools: ../../oauth2
"oauthInfo": {
"integration": "slack",
"token": "SLACK_TOKEN",
"scope": [
"userScope": [
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am changing this so that it is consistent with slack.

Copy link
Collaborator

@g-linville g-linville left a comment

Choose a reason for hiding this comment

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

This LGTM, but I think we should consider whether we want to split bot tools out into their own separate bundle. Also I think it would be good to add a tool to send messages to channels as a bot, rather than only DMs.

@StrongMonkey
Copy link
Contributor Author

@g-linville

This LGTM, but I think we should consider whether we want to split bot tools out into their own separate bundle. Also I think it would be good to add a tool to send messages to channels as a bot, rather than only DMs.

Yeah, I was thinking about pulling it as a separate bundle/tool too, wasn't super comfortable right now to have a separate slack tool to do thing as bot. But yeah we could have the discussion.

@StrongMonkey StrongMonkey merged commit ba9760a into obot-platform:main Feb 20, 2025
2 checks passed
@StrongMonkey
Copy link
Contributor Author

@g-linville I am merging this as a tool in existing bundle for now. Will consider adding message to channel later, this is more completing the demo use-case for unsubscribe agent.

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.

3 participants