-
Notifications
You must be signed in to change notification settings - Fork 24
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
Implement streaming API #13
Comments
Hi @lino-levan Thanks for your working on this project. About the update: i get it. we are saying the same thing. Looking forward to supporting streaming api. |
In the meantime, maybe consider removing the "stream" option from the API interface? Just wasted quite a bit of time thinking my code handling the stream was buggy, when really this library just crashes if you set |
I'll make a PR today. Sorry for the trouble! |
How is the progress? |
Sorry for the delay, didn't get around to doing it yesterday. PR is open now, waiting on a review from @load1n9 to merge. |
This uses
EventSource
, so we'll probably have to use Skye's polyfill until denoland/deno#14730 lands.The text was updated successfully, but these errors were encountered: