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

Use direct global binding for fetch in place of web_sys fetch bindings #474

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

jbms
Copy link
Contributor

@jbms jbms commented May 22, 2024

This avoids the need to detect whether the module is running in the main browser thread or in a WebWorker.

This matches how setInterval and related functions are handled by the gloo-timers crate.

See rustwasm/wasm-bindgen#3863

This avoids the need to detect whether the module is running in the main
browser thread or in a WebWorker.

This matches how `setInterval` and related functions are handled by the
gloo-timers crate.

See rustwasm/wasm-bindgen#3863
@littlebenlittle
Copy link
Contributor

littlebenlittle commented Jul 18, 2024

Confirmed with #486 that this is a fix for #484!

Is there any case where a context-specific fetch would be necessary?

Copy link
Collaborator

@ranile ranile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ranile ranile merged commit 7efcfe4 into rustwasm:master Jul 18, 2024
20 checks passed
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