<p></p>
<p><b>@chimp1984</b> approved this pull request.</p>

<p>utACK</p><hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/4896#discussion_r546458840">core/src/main/java/bisq/core/dao/governance/bond/lockup/LockupTxService.java</a>:</p>
<pre style='color:#555'>> -        Transaction txWithBtcFee = btcWalletService.completePreparedBsqTx(preparedTx, true, opReturnData);
+        Transaction txWithBtcFee = btcWalletService.completePreparedBsqTx(preparedTx, opReturnData);
</pre>
<p>It is the users responsibility and does not cause harm to others if he set a too low fee and need to resynce/redo the tx. I guess it was set to fee service rate initially but later changed... But yes, there might be arguments pro and con...</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/4896#discussion_r546458580">core/src/main/java/bisq/core/btc/wallet/BtcWalletService.java</a>:</p>
<pre style='color:#555'>> @@ -487,8 +499,6 @@ public Transaction completePreparedBsqTx(Transaction preparedBsqTx,
         int sigSizePerInput = 106;
         // typical size for a tx with 2 inputs
         int txVsizeWithUnsignedInputs = 203;
-        // If useCustomTxFee we allow overriding the estimated fee from preferences
-        Coin txFeePerVbyte = useCustomTxFee ? getTxFeeForWithdrawalPerVbyte() : feeService.getTxFeePerVbyte();
</pre>
<p>The name of useCustomTxFee was badly choosen. The intention was that for certain important transactions we want to use always the fee service (like in trade txs). I think in some BSQ txs that was the case earlier as well but got changed later as BSQ txs do not cause harm to other peers in case the fee was too low set.</p>
<p>So intention was to have an option to enforce usage of the fee service independent of what user has set in preferences. The false case would have done that. The true case will use whatever the user has set in preferences.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/bisq-network/bisq/pull/4896#pullrequestreview-556077510">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNU3BVYZDUKYUZWULMDSV2I73ANCNFSM4UNXH3MA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNS6JBOQH7UC7OQHE7DSV2I73A5CNFSM4UNXH3MKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOEESRDRQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bisq/pull/4896#pullrequestreview-556077510",
"url": "https://github.com/bisq-network/bisq/pull/4896#pullrequestreview-556077510",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>