Skip to content
This repository was archived by the owner on Nov 26, 2024. It is now read-only.

Commit 565cb2a

Browse files
committed
Remove fee_reason from SendToAddress
This is kruft from an earlier design; currently we only support v17 `sendtoaddress` which only responds with the txid.
1 parent d81952f commit 565cb2a

File tree

2 files changed

+1
-7
lines changed

2 files changed

+1
-7
lines changed

json/src/model/wallet.rs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,8 +83,6 @@ pub struct GetNewAddress(pub Address<NetworkUnchecked>);
8383
pub struct SendToAddress {
8484
/// The transaction id.
8585
pub txid: Txid,
86-
/// The transaction fee reason.
87-
pub fee_reason: String,
8886
}
8987

9088
/// Models the result of JSON-RPC method `gettransaction`.

json/src/v17/wallet.rs

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -146,11 +146,7 @@ impl SendToAddress {
146146
/// Converts version specific type to a version in-specific, more strongly typed type.
147147
pub fn into_model(self) -> Result<model::SendToAddress, hex::HexToArrayError> {
148148
let txid = self.0.parse::<Txid>()?;
149-
Ok(model::SendToAddress {
150-
txid,
151-
// FIXME: Is this acceptable?
152-
fee_reason: "".to_string(),
153-
})
149+
Ok(model::SendToAddress { txid })
154150
}
155151

156152
/// Converts json straight to a `bitcoin::Txid`.

0 commit comments

Comments
 (0)