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.

Sign up for GitHub

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

Jump to bottom

Browser Version #10

Closed
1wheel opened this issue Jul 12, 2020 · 2 comments
Closed

Browser Version #10

1wheel opened this issue Jul 12, 2020 · 2 comments

Comments

@1wheel
Copy link

1wheel commented Jul 12, 2020

This works great in the browser without the fetch import. I had to a do a little fiddling to figure that out and it looks @Fil did too.

Have you considered publishing the decoding function alone so this library can be used directly in the browser?
@j6k4m8
Copy link
Member

j6k4m8 commented Jul 13, 2020

I don't have that on my roadmap right now, but if you were interested in PR'ing a fix I'd love to review/merge :)

Is fetch giving you an error/failure or is it just unnecessary for your use? (I wanted to keep things server+browser-friendly but this may not be the right way to go about doing that.)

@1wheel 1wheel mentioned this issue May 17, 2021
Merged
@j6k4m8
Copy link
Member

j6k4m8 commented May 17, 2021

Closed in #14!

@j6k4m8 j6k4m8 closed this as completed May 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@j6k4m8 @1wheel