<p></p>
<blockquote>
<p>Once we are in a more final state we should think about the edge cases like when wallet is not synced.</p>
</blockquote>
<p>I've moved the precondition checks such that if they are not met then auto-confirmation is not started.</p>
<blockquote>
<p>.... also to define test use cases would help ourself for dev testing and later the testers for release testing.</p>
</blockquote>
<p>Here is my current list of test use cases.  I'll also put this into the main topic so it is more visible.</p>
<ul>
<li>New user (preferences)</li>
<li>Auto-confirm success</li>
<li>Premature manual confirm by user</li>
<li>XMR Tx Id / key not provided</li>
<li>XMR Tx not found</li>
<li>Re-used XMR TxId</li>
<li>Feature disabled in settings</li>
<li>Feature disabled by filter</li>
<li>Transaction limit exceeded</li>
<li>Invalid amount</li>
<li>XMR Trade date too old</li>
<li>XMR Trade date newer than Bisq trade date (ok)</li>
<li>Bisq shut down during Auto-confirmation procedure</li>
<li>XMR API service not reachable</li>
</ul>
<p>TBD:</p>
<blockquote>
<p>It would be nice to get some statistics how much the feature is used. We could get number of trades using the feature from the node operators (excluding those who run their own service).</p>
</blockquote>
<blockquote>
<p>We should consider to persist XmrProofResult in trade. for trade history it might be important to have some "proof" if autoconfirm was used or not, as well in dispute cases.<br>
if we use propbug for XmrProofResult the enum should not be used as enum in protobuf as protobuf enums are cumbersome (have global scope). use the name instead. see other enum serialisations...(ProtoUtil.enumFromProto)</p>
</blockquote>

<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/4421#issuecomment-682213233">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNVZUAPUDFWRZL7OIF3SC3JXHANCNFSM4QGSBOGQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNSBLUE5L3VUPF2CHHTSC3JXHA5CNFSM4QGSBOG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFCU364I.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/4421#issuecomment-682213233",
"url": "https://github.com/bisq-network/bisq/pull/4421#issuecomment-682213233",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>