Skip to content
This repository has been archived by the owner on Apr 18, 2024. It is now read-only.

feat: add stream chat api #112

Merged
merged 10 commits into from
Mar 17, 2023
Merged

feat: add stream chat api #112

merged 10 commits into from
Mar 17, 2023

Conversation

genffy
Copy link
Contributor

@genffy genffy commented Mar 12, 2023

issue: #86

stream api worked not as except, need some help to figure what's wrong.

if config runtime as edge, stream worked, but node:crypto not worked

  • add stream like api
  • use stream api at ChatRoom
  • add SimpleMarkdown to ChatRoom

some refs:
https://vercel.com/blog/gpt-3-app-next-js-vercel-edge-functions
vercel/next.js#9965 (comment)
https://beta.nextjs.org/docs/routing/route-handlers#streaming
https://beta.nextjs.org/docs/routing/route-handlers#edge-and-nodejs-runtimes
https://nextjs.org/docs/api-routes/edge-api-routes

@genffy genffy marked this pull request as draft March 12, 2023 15:32
@genffy genffy marked this pull request as ready for review March 14, 2023 04:31
@genffy genffy changed the title WIP: add stream chat api feat: add stream chat api Mar 14, 2023
@genffy genffy changed the title feat: add stream chat api WIP: add stream chat api Mar 14, 2023
@genffy
Copy link
Contributor Author

genffy commented Mar 14, 2023

still WIP, not merge @CGQAQ

@CGQAQ CGQAQ marked this pull request as draft March 14, 2023 07:03
@genffy genffy marked this pull request as ready for review March 15, 2023 05:04
@genffy genffy changed the title WIP: add stream chat api feat: add stream chat api Mar 15, 2023
@CGQAQ CGQAQ merged commit 414f8b1 into prompt-engineering:dev Mar 17, 2023
@CGQAQ
Copy link
Member

CGQAQ commented Mar 17, 2023

👍 LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants