LedgerHQ / lib-ledger-core

MIT License
79 stars 83 forks source link

VG-13274 - Confirmed-first ordering of utxos when crafting a transaction + add configuration #933

Closed jcoatelen-ledger closed 1 year ago

jcoatelen-ledger commented 1 year ago

Changes:

Testing

The new logic is tested with some new unit tests. To test the new behavior at integration level, prepare an account with utxos and remove the block_height value from most of the entries in DB, then craft a transaction. The expected result is that those with block_height should be proposed in priority.

References

github-actions[bot] commented 1 year ago

clang-tidy review says "All clean, LGTM! :+1:"