Page MenuHomePhabricator

[Cashtab] Get outputs for SLP v1 send txs
ClosedPublic

Authored by bytesofman on Jan 3 2024, 21:44.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Commits
rABCd5f3b2978f10: [Cashtab] Get outputs for SLP v1 send txs
Summary

New function to generate targetOutputs for slpv1 send txs

Test Plan

npm test

Diff Detail

Repository
rABC Bitcoin ABC
Branch
slp-send-helper-fn
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 26125
Build 51823: Build Diffcashtab-tests
Build 51822: arc lint + arc unit

Event Timeline

TODO

  • Add unit tests based on new txs
  • Review for where this function will fit in. Will you also need a function to determine tokenUtxos ? Where will this function be called? May want to combine this here, i.e. accept wallet, tokenId as params.

move check to after for loop

bytesofman published this revision for review.Jan 3 2024, 23:22

It's possible this function will change when I go to implement it, and work out the structure of how utxos will be selected for slpv1 transactions (mb ecash-coinselect is the best place to do this?). However, something very close to this function will be needed to deprecate the existing functions. So getting this in with the unit tests (based on broadcast slp txs) is a good starting point.

cashtab/package.json
41 ↗(On Diff #43859)

this upgrade is required to use the toLegacy function

This revision is now accepted and ready to land.Jan 4 2024, 08:47