diff options
| author | MeshCollider <[email protected]> | 2019-07-27 22:21:42 +1200 |
|---|---|---|
| committer | MeshCollider <[email protected]> | 2019-07-27 22:22:03 +1200 |
| commit | c606e6fc53f7630f82530cffe47900fa2128f27c (patch) | |
| tree | 61633dbe71b7124bedee4aed658d801412f6edf1 /src/interfaces/node.cpp | |
| parent | Merge #16301: Use CWallet::Import* functions in all import* RPCs (diff) | |
| parent | Add RPC bumpfee totalFee deprecation test (diff) | |
| download | discoin-c606e6fc53f7630f82530cffe47900fa2128f27c.tar.xz discoin-c606e6fc53f7630f82530cffe47900fa2128f27c.zip | |
Merge #15996: rpc: Deprecate totalfee argument in `bumpfee`
2f7eb772f6250442d4a0071318047cb2deeb31fa Add RPC bumpfee totalFee deprecation test (Jon Atack)
a92d9ce8cf355e18e43e1f207e4be9e42e7ec81a deprecate totalFee argument in bumpfee RPC call (Gregory Sanders)
Pull request description:
totalFee argument is of questionable use, and should be removed in favor of feerate-based features.
I first moved IsDeprecatedRPCEnabled because `bitcoin-wallet` doesn't link `libbitcoin_server`.
ACKs for top commit:
ryanofsky:
utACK 2f7eb772f6250442d4a0071318047cb2deeb31fa. Only change since last review is leaving IsDeprecatedRPCEnabled in its happy home, and switching to rpcEnableDeprecated instead. (Thanks!)
jonatack:
ACK 2f7eb772f6250442d4a0071318047cb2deeb31fa. Built locally, manually tested rpc bumpfee, help output ([gist](https://gist.github.com/jonatack/863673eacc02f9da39ff6d6712f9d837)), all tests pass. Travis failures appears to be unrelated, the [bitcoin builds are green](https://bitcoinbuilds.org/index.php?build=121).
meshcollider:
Code Review ACK 2f7eb772f6250442d4a0071318047cb2deeb31fa
Tree-SHA512: c97465205ee59575df37894bcbb6c4ecf8858dd8fe9d89503f9342b226768c1dcb553153bc9eb3055f7bf5eb41573e48b8efa57e083cd255793cbe5280f0026a
Diffstat (limited to 'src/interfaces/node.cpp')
0 files changed, 0 insertions, 0 deletions