wallet: add `outputs` argument to bumpfee and psbtbumpfee RPC for replacing outputs #25566

pull ghost wants to merge 1 commits into bitcoin:master from changing 6 files +76 −50

ghost commented at 6:46 PM on July 7, 2022: none

Fixes #22007 and alternative to #25344

Reviewers can test it with these commands:

$ bitcoin-cli -named sendtoaddress address="<address>" amount=<amount> replaceable=true

$ bitcoin-cli bumpfee <txid> '{"outputs": {"<new output>": <amout>}}'

No tests added yet.

DrahtBot added the label Wallet on Jul 7, 2022

wallet: add `outputs` argument to bumpfee and psbtbumpfee

`outputs` argument can be used to replace outputs in the replacement transaction when bumping fee

Co-Authored-By: rodentrabies <rodentrabies@protonmail.com>

84c11bc767

DrahtBot commented at 9:38 AM on July 8, 2022: contributor


The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Conflicts

Reviewers, this pull request conflicts with the following ones:
- #25344 (New extra_outputs argument for bumpfee/psbtbumpfee by rodentrabies)
- #23202 (wallet: allow psbtbumpfee to work with txs with external inputs by achow101)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
unknown marked this as a draft on Jul 8, 2022
rodentrabies commented at 5:16 PM on July 11, 2022: contributor

Changed #25344 to outputs instead of extra_arguments, so this can be discarded, I guess.
unknown closed this on Jul 11, 2022
bitcoin locked this on Jul 11, 2023

Contributors

Labels

Linked (view graph)

#22007 wallet: add destination (output) and bump fee #25344 New `outputs` argument for `bumpfee`/`psbtbumpfee`

wallet: add `outputs` argument to bumpfee and psbtbumpfee RPC for replacing outputs #25566

Conflicts