[bisq-network/bisq] PriceNode: Add support for multiple ExchangeRateProviders (#4315)

[sqrrm]

Thanks for the efforts on this PR. Rolling this out will need caution, but the main risks are usability. That could probably be helped by making sure the client handles faults better, as has been discussed.

I also have concerns regarding the loss of currencies. In particular the latin american ones seem to be missing. We should make an effort to not reduce usability in any market. Getting any market started is really tedious work and we should make sure to not lose any of that. I haven't read the code yet, but could we run this in parallel with BA for those currencies we don't yet have our own aggregation for?

-- 
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/4315#issuecomment-663499587
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20200724/7a34b32b/attachment.html>