Skip to content

Conversation

@jirispilka
Copy link
Contributor

@jirispilka jirispilka commented Oct 9, 2024

The documentation has two parts:

  • Real-time search data for OpenAI Assistant
  • Save data into OpenAI Vector Store and use it in the assistant

I know it's a bit dense, but I wanted to avoid splitting it into two separate pages

@TC-MO any idea where to add a full code example?
What about provide a link to github repository? Gist?

@github-actions github-actions bot added the t-ai Issues owned by the AI team. label Oct 9, 2024
@jirispilka jirispilka changed the title WIP: docs: Add open assistants integration docs: Add open assistants integration Oct 9, 2024
@jirispilka jirispilka marked this pull request as ready for review October 9, 2024 12:39
@jirispilka jirispilka requested a review from TC-MO as a code owner October 9, 2024 12:39
@TC-MO
Copy link
Contributor

TC-MO commented Oct 9, 2024

I think the best way even if it's huge would be to put the code example within the doc. As kind of a summary? If you are afraid that it super big maybe we can hide it in <details> tag?

Copy link
Contributor

@TC-MO TC-MO left a comment

Choose a reason for hiding this comment

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

Just one repetition that I found, otherwise looks good!

@jirispilka
Copy link
Contributor Author

Just one repetition that I found, otherwise looks good!

Oh, thanks a lot!

@jirispilka jirispilka merged commit 2b84dae into master Oct 10, 2024
7 checks passed
@jirispilka jirispilka deleted the docs/openai-assistants branch October 10, 2024 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-ai Issues owned by the AI team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants