Closed psolstice closed 4 months ago
The primary change involves the SendCoinsDialog::on_sendButton_clicked()
function in the QT interface of the wallet. Specifically, the update modifies the parameter passed to the CWallet::GetMinimumFee()
method from a hard-coded value of 0
to nTxConfirmTarget
, likely to use a more dynamic fee calculation based on transaction confirmation targets.
File Path | Change Summary |
---|---|
src/qt/sendcoinsdialog.cpp |
Modified the on_sendButton_clicked() function to change CWallet::GetMinimumFee() parameter from 0 to nTxConfirmTarget |
sequenceDiagram
participant User
participant SendCoinsDialog
participant CWallet
User ->> SendCoinsDialog: Click send button
SendCoinsDialog ->> CWallet: GetMinimumFee(nTxConfirmTarget)
CWallet -->> SendCoinsDialog: returns calculated fee
SendCoinsDialog ->> User: Displays confirmation dialog
🪙 In a line of code, a change so sleek,
The fee now answers tonTxConfirmTarget's
seek.
No more a zero stands its ground,
With dynamic fees, transactions bound,
Sending coins, swift and neat! 🚀From CodeRabbit, a little treat.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
PR intention
Calculate the fee for the second tx correctly