Add support for assistant.search.context#1667
Conversation
|
Thanks for the contribution! Before we can merge this, we need @ArielTM to sign the Salesforce Inc. Contributor License Agreement. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1667 +/- ##
==========================================
- Coverage 85.36% 85.20% -0.16%
==========================================
Files 113 113
Lines 12802 12829 +27
==========================================
+ Hits 10928 10931 +3
- Misses 1874 1898 +24 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Hi @ArielTM thanks for you contribution 🚀 they are alway welcome Unfortunately we need to hold off on introducing these changes due to these APIs being in a limited access stage
We would also need unit tests for these changes 🙏 |
|
@WilliamBergamin Until this is in "limited access" , would you say the best way to use this api would be to do something like this instead ? |
|
@Pythonista7 Yess this is the recommended approach 💯 |
Summary
Add support for assistant.search.context . Although listed in docs, it is yet to be supported.
There's a discrepancy between https://api.slack.com/methods/assistant.search.context and https://api.slack.com/docs/apps/data-access-api#use-action-token on whether
action_tokenis required. From my testing it is so I set it as such. Lmk if that's not the case.Testing
Ran it using my app and workspace
Category
/docs(Documents)/tutorial(PythOnBoardingBot tutorial)tests/integration_tests(Automated tests for this library)Requirements
python3 -m venv .venv && source .venv/bin/activate && ./scripts/run_validation.shafter making the changes.