[bisq-network/bisq] Fix issues with missing persistence for trade state (#4816)

[sqrrm]

I feel it might be cleaner to do the persist in the `complete()` method at the end of each trade task. It feels like that's what's being done with this PR anyway, but separately in each task with the chance of missing it somewhere.

Apart from that I think this looks ok and could be merged.

-- 
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/4816#issuecomment-730022166
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20201118/ae9d7375/attachment.html>