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

Support wasm target for download_progress #2419

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

skygrango
Copy link
Contributor

@skygrango skygrango commented May 2, 2024

I think it works, but I did make some tricky modifications because

  1. reqwest doesn't support chunk method for wasm, I have to replace chunk by stream
  2. the futures generated by wasm_bindgen NO Send, so I have to pin stream

螢幕截圖_20240502_171755

@skygrango skygrango force-pushed the wasm/support-download_progress branch from 62a0f05 to e4df484 Compare May 3, 2024 03:55
@skygrango skygrango force-pushed the wasm/support-download_progress branch from e4df484 to 5181677 Compare May 3, 2024 03:56
@skygrango skygrango changed the title Support wasm target for download_progress Support wasm target for download_progress May 3, 2024
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.

None yet

1 participant