[bisq-network/bisq] Add API 'gettrades' method (PR #5976)

Stan notifications at github.com
Wed Jan 26 14:52:20 CET 2022


> @ghubstan Everything seems to work now for my data set, but I think the response list should be sorted by date. BTW. do we have any limits option in our API endpoints regarding the max response entries? Might not be super critical ATM, but this could change in the future if liquidity changes and the amount of offers online. WDYT?

>> Response list should be sorted by date.
Yes, I should have sorted the lists by date.  I will include the sorting in this PR.

>> Any limits option in our API endpoints regarding the max response entries?
Not yet.  I will put it on my TODO list, and ask you later about what limits we should set.

By the way amigo, that's some _baaaaad_ data ;-)


-- 
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/5976#issuecomment-1022216551
You are receiving this because you are subscribed to this thread.

Message ID: <bisq-network/bisq/pull/5976/c1022216551 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20220126/ec17a1bb/attachment.htm>


More information about the bisq-github mailing list