edge-react-gui icon indicating copy to clipboard operation
edge-react-gui copied to clipboard

Action queue dryrun

Open samholmes opened this issue 3 years ago • 1 comments

PR Dependencies

https://github.com/EdgeApp/edge-react-gui/pull/3532 https://github.com/EdgeApp/edge-react-gui/pull/3526 https://github.com/EdgeApp/edge-react-gui/pull/3536

PR Requirements

If you have made any visual changes to the GUI. Make sure you have:

  • [ ] Tested on iOS device
  • [ ] Tested on Android device
  • [ ] Tested on small-screen device (iPod Touch)
  • [ ] Tested on large-screen device (tablet)

samholmes avatar Jul 26 '22 22:07 samholmes

After discussion with @swansontec, we've concluded what changes are needed:

  1. Implement the push HTTP API calls in this PR
  2. Include a new pendingTransactions: Uint8Array[] field in the EdgeSpendInfo for each call to makeSpend in order to "stack" non-broadcasted transactions. We'll need to hold pending transactions in the GUI's action queue state and implement the API upgrade in the currency plugins.

samholmes avatar Jul 29 '22 18:07 samholmes