Skip to content

add polyfill for readable stream async iterable support#7

Open
brycedorn wants to merge 1 commit intojonrhall:mainfrom
brycedorn:readable-stream-polyfill
Open

add polyfill for readable stream async iterable support#7
brycedorn wants to merge 1 commit intojonrhall:mainfrom
brycedorn:readable-stream-polyfill

Conversation

@brycedorn
Copy link
Copy Markdown

Hi! 👋 As pointed out in #6, the only browser that currently supports async iterable with ReadableStream is Firefox:

This adds the web-streams-polyfill package to fix behavior for all browsers. Confirmed working with Chrome 114.0.5735.90 and Safari 16.5.

@pdenya
Copy link
Copy Markdown

pdenya commented Jul 24, 2023

Any reason this isn't being merged? Is this repo being maintained?

@cesardgm
Copy link
Copy Markdown

I can vouch that @brycedorn 's commit solved streaming issues across different browsers. This commit should be merged. Otherwise, you will have to edit your local openai-streaming-hooks and link it to your app.

@pdenya
Copy link
Copy Markdown

pdenya commented Aug 16, 2023

This repo is not being maintained by @jonrhall. I switched to https://github.com/SpellcraftAI/openai-streams and have been happy with it

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.

3 participants