slack icon indicating copy to clipboard operation
slack copied to clipboard
verified publisher icon slack-go

support Data Access API

Open nktks opened this issue 6 months ago • 1 comments

  • support Data Access API
    • added https://api.slack.com/methods/assistant.search.context api
    • added assistant_thread attribute to Event API message, app_mention envets https://api.slack.com/docs/apps/data-access-api#event-listener
  • We are still testing this changes for Agent & AI Apps enabled app.
Pull Request Guidelines

These are recommendations for pull requests. They are strictly guidelines to help manage expectations.

PR preparation

Run make pr-prep from the root of the repository to run formatting, linting and tests.

Should this be an issue instead
  • [ ] is it a convenience method? (no new functionality, streamlines some use case)
  • [ ] exposes a previously private type, const, method, etc.
  • [ ] is it application specific (caching, retry logic, rate limiting, etc)
  • [ ] is it performance related.
API changes

Since API changes have to be maintained they undergo a more detailed review and are more likely to require changes.

  • no tests, if you're adding to the API include at least a single test of the happy case.
  • If you can accomplish your goal without changing the API, then do so.
  • dependency changes. updates are okay. adding/removing need justification.
Examples of API changes that do not meet guidelines:
  • in library cache for users. caches are use case specific.
  • Convenience methods for Sending Messages, update, post, ephemeral, etc. consider opening an issue instead.

nktks avatar Jun 04 '25 12:06 nktks

@nktks I'm not going to review this until you move it into ready for review - just wanted to call that out explicitly. Thanks for adding this.

nlopes avatar Jun 05 '25 09:06 nlopes

@nktks just checking in on this - no pressure at all but want to make sure you didn't just leave it at Draft by mistake :pray:

nlopes avatar Jun 28 '25 15:06 nlopes

@nlopes Sorry for the confusion. I've requested for the Data Access API to Slack, but it hasn't been approved yet for long time, so I'm still waiting.... Because of this, I haven't been able to test it....

nktks avatar Jul 01 '25 01:07 nktks

~Ah if that's the part that's missing, I can test it for you, no problem at all.~

Realised that developer sandbox environments don't have this enabled :(

nlopes avatar Jul 04 '25 16:07 nlopes

Yes. https://api.slack.com/docs/apps/data-access-api seems to be limited access stage. My company has joined Slack Enterprise Grid and I requested from https://api.slack.com/forms/data_access_api

nktks avatar Jul 08 '25 13:07 nktks

@nlopes Hello. Our app was approved and I could test thorough https://github.com/slack-go/slack/pull/1439/files#diff-03269bab64cd2cd7188b80de22221ee1321f6ee37f70d909a40e67205b724c24 .

So I changed as ready for review this PR. Could you please take a look when you have time?

nktks avatar Jul 09 '25 10:07 nktks