EdgeApp / edge-currency-monero

Javascript Monero currency plugin for Edge
BSD 3-Clause "New" or "Revised" License
5 stars 13 forks source link

William/fix send crash #66

Closed swansontec closed 2 years ago

swansontec commented 2 years ago

The bridgifyObject call is not working correctly, since we are using a different YAOB instance than the core.

Instead of dealing with this, we we send across the raw C++ API. Then we hijack the CppBridge object and instantiate it on the WebView side, using the C++ API we passed earlier. Yes, this is a hack, but it came out pretty cleanly.

paullinator commented 2 years ago

Wow you're right. That is a hack. Albeit a clever one. Took my awhile to understand what was going on and diffs don't make it clear w/o looking at react-native-mymonero-core