multi: add support for DCR-USDT pair on `/markets` view
- Add Mexc exchange
DCR-USDTpair. - Add Binance
DCR-USDTpair.
Breaking Changes:
1. `ratesproto`: Rate messages are no longer `DCR-BTC` only, a new `currencyPair` field indicates which market rate is sent.
`ExchangeSubscription` field changed from `btcIndex` to `index`.
2. `exchanges`: Aggregated chart data has been removed. This is because combining usdt and btc market bids and asks is not
ideal and serves very little purpose. Each market has its own chart.
3. Renamed three fields on `exchanges.ExchangeBotState`:
- `btc_index` -> `index`
- `dcr_btc_exchanges` -> `dcr_exchanges` (this field now returns a nested map of supported markets)
- `btc_indices` -> `indices` (this field now returns a nested map of supported indices)
4. Affected API Endpoints:
- `/exchangerate` (`exchanges.ExchangeRates.Exchanges` json field returns a nested map of support markets)
- `/exchanges` (returns modified `exchanges.ExchangeBotState`)
Closes #2001
This PR sets the foundation for adding other pairs and exchanges. As per UI, we can always improve on that if there are suggestions. If a redesign is want we want, I can get that sorted.
I can't speak for others, but the aggregate chart is kinda the whole point for me. Can't we just convert all depth to USD?
I can't speak for others, but the aggregate chart is kinda the whole point for me. Can't we just convert all depth to USD?
I'm expecting a review from @davecgh.
@davecgh what do you think?
Thank you for the review @davecgh.
I agree with @buck54321 in that I very much like the aggregated chart and find it much more useful overall.
I'd like to work on this in another PR..let this one remain focused. @buck54321, @davecgh cool?
I'd like to work on this in another PR..let this one remain focused. @buck54321, @davecgh cool?
That sounds fine to me.