Skip to content
This repository has been archived by the owner on Feb 15, 2022. It is now read-only.

Fix for 400 bad request from Binance #1256

Merged
merged 1 commit into from
Feb 1, 2018

Conversation

aquamanjl
Copy link
Contributor

CCXT lib update forced this change (release 1.10.817). Fixes 1254

@Shawn8901
Copy link
Contributor

I think we are also not inserting a correct since value. Can someone check this?

@DeviaVir DeviaVir merged commit cdf4fd5 into DeviaVir:unstable Feb 1, 2018
@aquamanjl
Copy link
Contributor Author

@Shawn8901 Not sure how the date range is working but it seems to be passed correctly. Here is what I see when running backfill with different days on unstable branch

$ ./zenbot.sh backfill binance.ETH-BTC --days 1

binance.ETH-BTC saved 7479 trades 1 days left
..^C
$ ./zenbot.sh backfill binance.ETH-BTC --days 3

binance.ETH-BTC saved 12028 trades 3 days left
^C
$

Heerpa pushed a commit to Heerpa/zenbot that referenced this pull request Feb 25, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants