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

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/4559#discussion_r499142492">common/src/main/java/bisq/common/taskrunner/Task.java</a>:</p>
<pre style='color:#555'>> -        taskHandler.handleErrorMessage(errorMessage);
+        taskHandler.handleErrorMessage(t.getMessage());
</pre>
<p>I would like to exclude this change from this PR. It would affect the UI notifications that depend on the errormessage being reasonable for display.</p>
<p>Generally refactoring is fine (although more care is needed now with the many big PRs coming in, to avoid conflicts) and behavioral changes are fine as long as the ramifications are taken into account, I feel that's lacking here.</p>
<p>That's the only issue I have left. The rest looks good.</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/4559#pullrequestreview-501541716">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNWPFH7FPYJE4ZBSHULSI4H6ZANCNFSM4RXT56GA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNTTXCP5X3Q66GMCVRDSI4H6ZA5CNFSM4RXT56GKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGODXSOWVA.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/4559#pullrequestreview-501541716",
"url": "https://github.com/bisq-network/bisq/pull/4559#pullrequestreview-501541716",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>