altangent / ccxws

WebSocket client for 38 cryptocurrency exchanges
MIT License
619 stars 186 forks source link

Do we really need id for candles #308

Closed sribna closed 2 years ago

sribna commented 2 years ago
let market = {
                    id: 'BTCUSDT',
                    base: 'BTC',
                    quote: 'USDT'
                }
client.subscribeCandles(market);

The ID is required, although it can be optional and constructed automatically per exchange from the base and quote. In my case, I had to add an extra field to the database table + some logic to handle it. I'm not complaining of course, just saying that it would be easier for developers to have it in ccxws's handlers