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

feature request: add Blob and File support in 2.* #61

Closed
117503445 opened this issue Nov 28, 2023 · 2 comments · Fixed by #62
Closed

feature request: add Blob and File support in 2.* #61

117503445 opened this issue Nov 28, 2023 · 2 comments · Fixed by #62
Labels

Comments

@117503445
Copy link

117503445 commented Nov 28, 2023

In version 1.0.1, support for Blob and File types was available. However, it seems that support for Blob and File types has been removed in version 2.0.0 after the introduction of TypeScript. The error message states, Type 'function Blob() { [native code] }' not yet supported.

Now that PeerJS is using the 2.0.0 version of js-binarypack, I would like to request the addition of support for Blob and File types in version 2.*.

@LetMeDream
Copy link

Hello there. Encountering the same issue on my side. Would really appreciate it if the feature could be added or if there's some workaround for it

jonasgloning added a commit that referenced this issue Dec 3, 2023
`pack` will return a `Promise` when it packs a `Blob`
Switching to `bun test`. `jsdom` is annoying.

Closes #61
jonasgloning added a commit that referenced this issue Dec 3, 2023
`pack` will return a `Promise` when it packs a `Blob`.

Closes #61
github-actions bot pushed a commit that referenced this issue Dec 3, 2023
# [2.1.0](v2.0.0...v2.1.0) (2023-12-03)

### Features

* re-add Blob support ([cd17d63](cd17d63)), closes [#61](#61)
* re-add Blob support ([#62](#62)) ([e79d1ee](e79d1ee)), closes [#61](#61)
@jonasgloning
Copy link
Member

🎉 This issue has been resolved in version 2.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging a pull request may close this issue.

3 participants