[bisq-network/bisq] Trade protocol domain improvements (#4566)

sqrrm notifications at github.com
Tue Sep 29 13:34:19 UTC 2020


@sqrrm commented on this pull request.



>  @Slf4j
-public class SellerSendsDepositTxAndDelayedPayoutTxMessage extends TradeTask {
-    @SuppressWarnings({"unused"})
-    public SellerSendsDepositTxAndDelayedPayoutTxMessage(TaskRunner taskHandler, Trade trade) {
+public class SellerSendsDepositTxAndDelayedPayoutTxMessage extends SendMailboxMessageTask {

Then I'll drop it. Main reason was to encapsulate the logic to not have to think about it twice. If we do one more resend message it will likely be worth it.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/4566#discussion_r496720197
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20200929/60ce54b6/attachment.html>


More information about the bisq-github mailing list