Rebased on #15557, review 49dfa31 only.
wallet: Keep all outputs in bumpfee #15656
pull promag wants to merge 4 commits into bitcoin:master from promag:2019-03-bumpall changing 7 files +248 −45-
promag commented at 7:37 PM on March 23, 2019: member
-
generalize bumpfee to add inputs when needed 90c33cdc96
-
add functional tests for feerate bumpfee with adding inputs be2d4356d3
-
wallet_bumpfee.py: add test for change key preservation 8e9027bd98
-
wallet: Keep all outputs in bumpfee 49dfa31d46
-
promag commented at 7:46 PM on March 23, 2019: member
This was suggested in @instagibbs PR #15557 (review)
-
DrahtBot commented at 10:37 PM on March 23, 2019: member
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #15557 (Enhance
bumpfeeto include inputs when targeting a feerate by instagibbs) - #15157 (rpc: Bumpfee units change, satoshis to BTC by benthecarman)
- #14641 (rpc: Add min_conf option to fund transaction calls by promag)
- #12096 ([rpc] [wallet] Allow specifying the output index when using bumpfee by kallewoof)
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.
- #15557 (Enhance
- fanquake added the label Wallet on Mar 23, 2019
-
promag commented at 2:05 PM on April 14, 2019: member
See #15557 (review).
- promag closed this on Apr 14, 2019
- promag deleted the branch on Apr 14, 2019
- DrahtBot locked this on Dec 16, 2021
Labels