<p></p>
<p><b>@chimp1984</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/4486#discussion_r484170917">core/src/main/java/bisq/core/trade/protocol/TradeProtocol.java</a>:</p>
<pre style='color:#555'>> @@ -420,4 +420,14 @@ private void cleanupTradableOnFault() {
             }
         }
     }
+
+    protected boolean wasDisputed(ErrorMessageHandler errorMessageHandler) {
+        if (trade.getDisputeState() != Trade.DisputeState.NO_DISPUTE) {
+            String msg = "Dispute have been opened once. We do not allow anymore to confirm payment by button click.";
+            log.error(msg);
</pre>
<p>I think that is not too harsh as it is not possible to get there as long a user does not write custom code. Will be also more visible in case the API starts to use those code branches and does not do the checks we do in the UI. Maybe even an exception is justified here?</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/4486#discussion_r484170917">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNSTHNUTYJGO7OTFVW3SERK67ANCNFSM4Q3SP7IQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNUZMUPEQSOOIXWL74DSERK67A5CNFSM4Q3SP7I2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGODTGXP7A.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/4486#discussion_r484170917",
"url": "https://github.com/bisq-network/bisq/pull/4486#discussion_r484170917",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>