feat: Slack AI & Assistants Compatibility #1331
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
This PR resolves issue 1326. |
There was a problem hiding this comment.
Hey @MattDavisRV , thanks for the PR! I've got some minor feedback items I added.
It also looks like you may have included some new events, which while very much appreciated, may not be related to the Assistant Events.
Could you please submit these changes in a different PR so we can keep PRs scoped to a single issue or share the documentation where they are required for the Assistant functionality?
lorenzoaiello
added
the
feedback given
…ram, not currently used by Slack
|
@lorenzoaiello, sorry about the initially cluttered PR, I didn't look at the changed files and I'm unsure how all of the additional changes were added in. I've cleaned up the PR to only be my changes. I've also added in an additional change for the Section Block that was added due to the AI changes. I was going to submit an additional PR with it, but I decided to add it in here. |
4 tasks
|
@MattDavisRV , can you please resolve the conflicts? |
|
@lorenzoaiello all set, I refactored my tests, it looks like i can't use a handler function with form variables, I noticed other tests were using the standard "ok handler", so I switched to that. All tests are passing now. |
lorenzoaiello
removed
the
feedback given
|
Hey @lorenzoaiello, anything else I need to do here to get this merged in? |
lorenzoaiello
approved these changes
Nov 6, 2024
lorenzoaiello
changed the title
lorenzoaiello
pushed a commit
that referenced
this pull request
Nov 12, 2024
This is a fix for [PR 1331](#1331), which I mistakenly removed the channel_id parameter from the API request during feedback. Without channel_id provided, the call always returns `invalid_arguments`.
gautamr95
pushed a commit
to pylon-labs/slack
that referenced
this pull request
Dec 13, 2024
Added new function for Slack AI & Assistants, information can be found here: https://api.slack.com/docs/apps/ai New Events: https://api.slack.com/events/assistant_thread_started https://api.slack.com/events/assistant_thread_context_changed New API Calls: https://api.slack.com/methods/assistant.threads.setStatus https://api.slack.com/methods/assistant.threads.setSuggestedPrompts https://api.slack.com/methods/assistant.threads.setTitle Tests have been added and pass all checks via `make pr-prep`.
gautamr95
pushed a commit
to pylon-labs/slack
that referenced
this pull request
Dec 13, 2024
…k-go#1344) This is a fix for [PR 1331](slack-go#1331), which I mistakenly removed the channel_id parameter from the API request during feedback. Without channel_id provided, the call always returns `invalid_arguments`.
gautamr95
pushed a commit
to pylon-labs/slack
that referenced
this pull request
Dec 13, 2024
Added new function for Slack AI & Assistants, information can be found here: https://api.slack.com/docs/apps/ai New Events: https://api.slack.com/events/assistant_thread_started https://api.slack.com/events/assistant_thread_context_changed New API Calls: https://api.slack.com/methods/assistant.threads.setStatus https://api.slack.com/methods/assistant.threads.setSuggestedPrompts https://api.slack.com/methods/assistant.threads.setTitle Tests have been added and pass all checks via `make pr-prep`.
gautamr95
pushed a commit
to pylon-labs/slack
that referenced
this pull request
Dec 13, 2024
…k-go#1344) This is a fix for [PR 1331](slack-go#1331), which I mistakenly removed the channel_id parameter from the API request during feedback. Without channel_id provided, the call always returns `invalid_arguments`.
gautamr95
pushed a commit
to pylon-labs/slack
that referenced
this pull request
Dec 13, 2024
Added new function for Slack AI & Assistants, information can be found here: https://api.slack.com/docs/apps/ai New Events: https://api.slack.com/events/assistant_thread_started https://api.slack.com/events/assistant_thread_context_changed New API Calls: https://api.slack.com/methods/assistant.threads.setStatus https://api.slack.com/methods/assistant.threads.setSuggestedPrompts https://api.slack.com/methods/assistant.threads.setTitle Tests have been added and pass all checks via `make pr-prep`.
gautamr95
pushed a commit
to pylon-labs/slack
that referenced
this pull request
Dec 13, 2024
…k-go#1344) This is a fix for [PR 1331](slack-go#1331), which I mistakenly removed the channel_id parameter from the API request during feedback. Without channel_id provided, the call always returns `invalid_arguments`.
gautamr95
pushed a commit
to pylon-labs/slack
that referenced
this pull request
Dec 13, 2024
Added new function for Slack AI & Assistants, information can be found here: https://api.slack.com/docs/apps/ai New Events: https://api.slack.com/events/assistant_thread_started https://api.slack.com/events/assistant_thread_context_changed New API Calls: https://api.slack.com/methods/assistant.threads.setStatus https://api.slack.com/methods/assistant.threads.setSuggestedPrompts https://api.slack.com/methods/assistant.threads.setTitle Tests have been added and pass all checks via `make pr-prep`.
gautamr95
pushed a commit
to pylon-labs/slack
that referenced
this pull request
Dec 13, 2024
…k-go#1344) This is a fix for [PR 1331](slack-go#1331), which I mistakenly removed the channel_id parameter from the API request during feedback. Without channel_id provided, the call always returns `invalid_arguments`.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added new function for Slack AI & Assistants, information can be found here: https://api.slack.com/docs/apps/ai
New Events:
https://api.slack.com/events/assistant_thread_started
https://api.slack.com/events/assistant_thread_context_changed
New API Calls:
https://api.slack.com/methods/assistant.threads.setStatus
https://api.slack.com/methods/assistant.threads.setSuggestedPrompts
https://api.slack.com/methods/assistant.threads.setTitle
Tests have been added and pass all checks via
make pr-prep.