[bisq-network/bisq] Purge saved trades that failed market price check (#2930)

sqrrm notifications at github.com
Sun Jun 30 09:00:44 UTC 2019


@devinbileck I agree that building from source is probably too hard for a lot of users. We could build this branch for them as well, like the main release.

Having a separate tool would be perhaps more correct but I don't really see many benefits. This fix is very specific to repair the data dir for a bug that has now been fixed so it's a one time use. The separate tool would have to be maintained just as much as bisq itself if it were to be reused and we would have to build it for different platforms as well. To add any task that users might want to perform regularly in a separate tool doesn't make sense to me. Why wouldn't we just implement that in core to never have the issue in the first place?

-- 
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/2930#issuecomment-507020211
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20190630/a18fa498/attachment.html>


More information about the bisq-github mailing list