search

webcerebrium / java-binance-api

Java Binance API Client
MIT License
73 stars 40 forks source link

BinanceApi::klines() and BinanceApi::aggTrades() seem buggy #11

Closed Creckle closed 6 years ago

Creckle commented 6 years ago

https://github.com/webcerebrium/java-binance-api/blob/db3378ef03c1e8752b6cee13087b1e5a3395a732/src/main/java/com/webcerebrium/binance/api/BinanceApi.java#L232

change 'or' to 'and' same goes for https://github.com/webcerebrium/java-binance-api/blob/db3378ef03c1e8752b6cee13087b1e5a3395a732/src/main/java/com/webcerebrium/binance/api/BinanceApi.java#L181 and the next line

wcrbrm commented 6 years ago

@Creckle thanks for your feedback.

wcrbrm commented 6 years ago

the fix was included in Release 1.0.8, just pushed it on Maven