MutinyWallet / mutiny-web-poc

Mutiny node frontend proof of concept
MIT License
6 stars 3 forks source link

Move to mutiny-wasm #39

Closed TonyGiorgio closed 1 year ago

TonyGiorgio commented 1 year ago

Needs to be updated to new version of mutiny-core/wasm once this is merged and released: https://github.com/MutinyWallet/mutiny-node/pull/371

cloudflare-pages[bot] commented 1 year ago

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6e5379a
Status: ✅  Deploy successful!
Preview URL: https://2811bedf.mutiny-web-poc-reckless.pages.dev
Branch Preview URL: https://mutiny-wasm.mutiny-web-poc-reckless.pages.dev

View logs

benthecarman commented 1 year ago

Think there should be a few compile issues, when you build, I have something locally I can push up

TonyGiorgio commented 1 year ago

It's probably just because of the changes I needed in https://github.com/MutinyWallet/mutiny-node/pull/371 - will get that fixed whenever we do another release.

futurepaul commented 1 year ago

Screenshot 2023-04-17 at 12 15 40 PM

seems like the associated methods aren't in the wasm package perhaps?

futurepaul commented 1 year ago

oh my bad I didn't see the comments above

benthecarman commented 1 year ago

something seems wrong, this parameter isn't appearing either

https://github.com/MutinyWallet/mutiny-node/blob/master/mutiny-wasm/src/lib.rs#L210

TonyGiorgio commented 1 year ago

Hmm, I'll double check when the release is built and deployed. Worked when I pointed it to my local wasm lib.

TonyGiorgio commented 1 year ago

Works now: https://2811bedf.mutiny-web-poc-reckless.pages.dev/