diff --git a/CHANGELOG.md b/CHANGELOG.md index 8ae2ff95..06bcf981 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -10,7 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 - Added Bitcoin core RPC added as blockchain backend - Added a `verify` feature that can be enable to verify the unconfirmed txs we download against the consensus rules -- Removed and replaced `set_single_recipient` with more general `allow_shrinking`. +- Removed and replaced `set_single_recipient` with more general `drain_to` and replaced `maintain_single_recipient` with `allow_shrinking`. ## [v0.8.0] - [v0.7.0] diff --git a/src/wallet/mod.rs b/src/wallet/mod.rs index 29f43087..ba0ab8e0 100644 --- a/src/wallet/mod.rs +++ b/src/wallet/mod.rs @@ -3128,7 +3128,7 @@ pub(crate) mod test { // them, and make sure that `bump_fee` doesn't try to add more. This fails because we've // told the wallet it's not allowed to add more inputs AND it can't reduce the value of the // existing output. In other words, bump_fee + manually_selected_only is always an error - // unless you've also set "allow_shrinking OR there is a change output". + // unless you've also set "allow_shrinking" OR there is a change output. let incoming_txid = crate::populate_test_db!( wallet.database.borrow_mut(), testutils! (@tx ( (@external descriptors, 0) => 25_000 ) (@confirmations 1)), @@ -3307,7 +3307,7 @@ pub(crate) mod test { Some(100), ); - // initially make a tx without change by using `set_drain_recipient` + // initially make a tx without change by using `drain_to` let addr = Address::from_str("2N1Ffz3WaNzbeLFBb51xyFMHYSEUXcbiSoX").unwrap(); let mut builder = wallet.build_tx(); builder diff --git a/src/wallet/tx_builder.rs b/src/wallet/tx_builder.rs index 20ea1955..3ee2cd71 100644 --- a/src/wallet/tx_builder.rs +++ b/src/wallet/tx_builder.rs @@ -569,9 +569,8 @@ impl<'a, B, D: BatchDatabase, Cs: CoinSelectionAlgorithm> TxBuilder<'a, B, D, /// difference is that it is valid to use `drain_to` without setting any ordinary recipients /// with [`add_recipient`] (but it is perfectly find to add recipients as well). /// - /// When bumping the fees of a transaction made with this option, the user should remeber to - /// add [`allow_shrinking`] to correctly update the - /// single output instead of adding one more for the change. + /// When bumping the fees of a transaction made with this option, you probably want to + /// use [`allow_shrinking`] to allow this output to be reduced to pay for the extra fees. /// /// # Example ///