River Kanies

@DanGould should I do the `target_family = "wasm", target_os = "unknown"` stuff or is just a `web` feature enough for now? I was thinking just make a web feature and...

interesting but WASM works in ReactNative and Node.js envs aswell, and it looks like this doesn't solve for that(?). On the bindings side I'm using `web_sys::js_sys::Date` and I know that...

I know MetaMask is using bdk-wasm in ReactNative, so it's just an option for that we have available now. Yeah it'd be great to have swift and kotlin bindings and...

https://github.com/payjoin/rust-payjoin/pull/596 replaces this hopefully, so we can close this pr I suppose. want to test ReactNative hopefully soon

On further discussion with Dan this approach is still under consideration. The reason I originally proposed this was that it's the approach bdk takes; however, looking into it I don't...

Steve: """ time is needed to so the bdk_chain crate knows how to order the blockdata, the apply_update_at function was added for no_std environments like wasm that don't have access...

@DanGould ive come back around to thinking this is the best approach. Lowest maintenance, no extra deps, highest flexibility. I confirmed that this was the reasoning behind bdk approach to...

What do you mean "as in 596"? That's a totally different approach which I'm arguing is not the way to go. It's much more complex and web specific

The reason I opted to use systemTime here is that is what you get from JS-sys, how I had it set up was to expose only new in bindings but...

@DanGould here is the commit that shows how the interface is used: https://github.com/LtbLightning/payjoin-ffi/pull/49/commits/f78480210297d09f364294cb3ffea2a6fdd3be3e Since js_sys::Date::now() returns a u64 I add it to the std::time::SystemTime::UNIX_EPOCH at the bindings level to get...