Skip to content

Conversation

@bennycode
Copy link
Contributor

I updated the payload with a real-world example which I logged. The payload now contains the additional updateTime and cummulativeQuoteQty which are part of the Binance API:

In general we should consider linking to Binance's response examples (or the definitions of our index.d.ts file) instead of maintaining our own payloads in a Markdown file.

@balthazar
Copy link
Collaborator

It's more of a convenience thing for users as the data is reorganized for some endpoints, but I agree it's a pain to update since the API changes often still

@coveralls
Copy link

Coverage Status

Coverage decreased (-26.5%) to 63.971% when pulling ebff4aa on bennyn:patch-3 into 6197719 on HyperCubeProject:master.

@balthazar balthazar merged commit e550c43 into ccxt:master Jan 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants