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

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/3044#discussion_r315679361">core/src/main/java/bisq/core/offer/OpenOfferManager.java</a>:</p>
<pre style='color:#555'>> @@ -97,6 +98,8 @@
     private final P2PService p2PService;
     private final BtcWalletService btcWalletService;
     private final TradeWalletService tradeWalletService;
+    @Inject
+    private TradeManager tradeManager;
</pre>
<p>well, that it not so easy. It is definitely not solvable with best-practice Java. Here is some food-for-thought:<br>
<a target="_blank" rel="noopener noreferrer" href="https://user-images.githubusercontent.com/1070734/63349721-472c3d80-c35c-11e9-8300-a2f417ca8a70.png"><img src="https://user-images.githubusercontent.com/1070734/63349721-472c3d80-c35c-11e9-8300-a2f417ca8a70.png" alt="Screenshot from 2019-08-20 15-06-27" style="max-width:100%;"></a><br>
(black is status, green works out of the box, red requires dependency, the grey thought experiment introduces 2 circular dependencies). All in all, some devilish trickery seems due.</p>
<p>Or: What if we only check for EOL hidden services on trade complete? Bisq is about trading and not about cancelling offers. The HS might live longer without being needed but does that hurt? Eventually a trade completes and thus, the HS is cleaned up.</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/3044?email_source=notifications&email_token=AJFFTNXHQTJXWOXFPZXPHPLQFPUT5A5CNFSM4IIPXYQ2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCCCQRNY#discussion_r315679361">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNU4EIJP5WQCR6AKM4TQFPUT5ANCNFSM4IIPXYQQ">mute the thread</a>.<img src="https://github.com/notifications/beacon/AJFFTNTRHNHLOGK2NIME2STQFPUT5A5CNFSM4IIPXYQ2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCCCQRNY.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/3044?email_source=notifications\u0026email_token=AJFFTNXHQTJXWOXFPZXPHPLQFPUT5A5CNFSM4IIPXYQ2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCCCQRNY#discussion_r315679361",
"url": "https://github.com/bisq-network/bisq/pull/3044?email_source=notifications\u0026email_token=AJFFTNXHQTJXWOXFPZXPHPLQFPUT5A5CNFSM4IIPXYQ2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCCCQRNY#discussion_r315679361",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>