Commit Graph

9 Commits (master)

Author SHA1 Message Date
Paul Shapiro 3073a0e81c c 2018 -> 2019
5 years ago
Paul Shapiro c1c812bf64 updated monero_utils references to coreBridge_instance names for clarity - only mymonero-app-js needs to have a 'monero_utils' bridge at this point, so library consumers can now think about using MyMoneroCoreBridge directly (hurray)
6 years ago
Paul Shapiro 9bfa1acee0 big changes: removed monero_utils file, plus IPCSafe_remote_monero_utils (this is now up to applications); completely reworked monero_sendingFunds - SendFunds is gone, now implemented in MyMoneroCoreBridge; removed response_parser_utils unspent outs parsing; updated sendingFunds.spec.... next commit will change CoreBridge return { err_msg } to throws since no IPC is necessary anymore
6 years ago
Paul Shapiro 789c1fa71b breaking change to UnspentOuts hostedMoneroAPIClient method callback args: removed redundant unusedOutputs and switched over to mandatory return of per byte fee - prefers per b in API response with fallback to per kb for deploy rollover - per kb can be removed in the future
6 years ago
Paul Shapiro 5611835d14 updated mymonero-core-cpp; updated bridge for new stepwise sending implementation (ported majority of SendFunds to C++ as send_step*_…; updated tests and removed removed bridge methods like calculate_fee, estimate rct tx size, and create_transaction; New: added tests/sendingFunds.spec.js as integration tests for new monero_sendingFunds_utils implementation - run with 'npm test -- tests/sendingFunds.spec.js'; CMakeLists: enabled ASSERTIONS
6 years ago
Paul Shapiro ef830ad28a lots of moving things around; implemented promise constructor for monero_utils (which is actually MyMoneroCoreBridge, a new class to replace cnUtil).. this was chosen after trying both promises directly in the functions and async/await in one or two decent patterns in order to avoid the requirement that integrators support async/await and to retain the option for calls to monero_utils to remain synchronous - added src/module-post as a post-js in order to fix emscripten .then infinite loop ... this PR does not include any kind of support for calling from electron renderer to electron main - that is, the code to detect electron renderer, then intercept ret.err_msg and throw only if not in electron renderer, and relay ret.err_msg otherwise has been removed for the moment.. will likely be added back
6 years ago
Paul Shapiro 902a35253c updated MyMoneroCoreCpp.js build; cryptonote_utils: extracted destination formatting and money amount formatting to sendingFunds_utils and money_format_utils respectively, and added IPCsafe (string instead of JSBigInt params) create_transaction variant to support call from electron renderer procs etc, and added more complete load readiness waiting interface, and added locateFile impl for passing as emscripten Module template to locate wasm file in various environments; monero_cryptonote_utils_instance: detecting electron renderer and exposing via electron.remote call (will this cause issues with anyone's build systems trying to include electron\?); commenting logs
6 years ago
HenryNguyen5 6f3142aa1c Add prettier support
6 years ago
Paul Shapiro 331398f39d monero_utils/mymonero_response_parser_utils.js -> hostAPI/response_parser_utils.js; added hostAPI/net_service_utils.js, holding HostedMoneroAPIClient_Base request implementation - still not 100% sure it's right to move the actual get unspent outs, get rand outs, and tx submit request calls and response parsing calls to sendingFunds_utils, primarily because parsing may be wrapped in an application-level abstraction for platform-dependent concurrency
6 years ago