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

Make streaming vercel compatible #62

Closed
wants to merge 7 commits into from
Closed

Make streaming vercel compatible #62

wants to merge 7 commits into from

Conversation

WonderPG
Copy link
Contributor

No description provided.

@jankrepl
Copy link
Collaborator

Thank you for doing this @WonderPG

Note that for the this to be compatible with the vercel UI SDK more work needs to be done

  1. A special header needs to be inserted: https://github.com/vercel/ai/blob/main/examples/next-fastapi/api/index.py#L134C5-L134C55
  2. Different request body https://github.com/vercel/ai/blob/main/examples/next-fastapi/api/index.py#L22-L23

I know I know, this is a lot of changes. I am fine with this entire vercel compatibility being just a side project and we don't have to merge to main if we don't like it. We can work on the above together.

@jankrepl
Copy link
Collaborator

We moved this to a different repo. Closing.

@jankrepl jankrepl closed this Jan 23, 2025
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.

2 participants