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

Add wrapper for wasm client and mix-fetch in nodejs #4006

Merged
merged 4 commits into from
Nov 1, 2023

Conversation

sebastinez
Copy link
Contributor

@sebastinez sebastinez commented Oct 17, 2023

Description

Adds a NodeJS version for mix-fetch.
Adds a NodeJS wrapper for running a nym WASM client.

  • Uses the existing SDK worker and a wasm packed for NodeJS.

This PR doesn't provide a full-fat version yet, since wasm-pack has some quirks on how the wasm bytes are loaded for --target node, eventually I'll be able to add it to this PR.

Checklist:

  • added a changelog entry to CHANGELOG.md

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you for making this first PR

@mmsinclair
Copy link
Contributor

@sebastinez sebastinez force-pushed the sebastinez/nodejs-wasm-client branch 2 times, most recently from 363e1ce to 95cb7bd Compare October 19, 2023 05:24
@sebastinez sebastinez changed the title Add nodejs wrapper for wasm client Add wrapper for wasm client and mix-fetch in nodejs Oct 19, 2023
@sebastinez sebastinez force-pushed the sebastinez/nodejs-wasm-client branch 6 times, most recently from a60cb9b to e3938a9 Compare October 24, 2023 08:01
@sebastinez sebastinez force-pushed the sebastinez/nodejs-wasm-client branch 2 times, most recently from b590f0b to 947c915 Compare October 31, 2023 12:58
@sebastinez sebastinez merged commit 06a96fa into develop Nov 1, 2023
18 of 19 checks passed
@sebastinez sebastinez deleted the sebastinez/nodejs-wasm-client branch November 1, 2023 08:31
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.

2 participants