forked from DeviaVir/zenbot
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge branch kraken_fixes #1
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* CEXIO error handling also fixes `/home/user/zenbot/lib/engine.js:291 order.status = api_order.status` crash. * Update exchange.js
* On branch master Your branch is up-to-date with 'origin/master'. Changes to be committed: modified: conf-sample.js modified: extensions/exchanges/wexnz/exchange.js modified: package-lock.json modified: package.json ========================================================= package.json: bumped wexnz npm module from 1.0.2 to 1.0.3 to retain v2 api settings for now since our 1.0.2 started at v3, but work will need to be done on a new exchange.js for v3. ========================================================= package-lock.json: routine package-lock.json update for 1.0.3 npm module. ========================================================= conf-sample.js: added a note notifying wex.nz traders that backfilling might not support long-hauls until v2 api limits are raised from 5000 max trades or the tradeHisotry function can be fiured out..... Other exchanges just use "trades" though. ========================================================= exchange.js: A few things, set rate limit to 1, trades count to something over ~9000 to return as many trades as the api possibly can so that the duplicate trade error does not re-occurr at smaller backfills (such as --days=0.1) and while trading at 5000 min periods. exchange.js: Also set authed_client() typo to authedClient() as it should be. ========================================================= Let me know if there is anything else that can be done for now, if any errors occur. I grabbed an API key, it should authenticate fine. @sinister has been authenticating, but might have bugs buying/selling at the moment. Please reply here for any bugs, and I will try my best to iron them out in a few days. JM ~ Sun 03 Dec 2017 05:44:08 AM CST * Increased rate limit * To account for BTC future prices, increased number format digits. To account for BTC future prices, increased number format digits. * Moved .toLowerCase up * forgot parenthesis, moved .toLowerCase() UP forgot parenthesis, moved .toLowerCase() UP * Pushed empty col for better spacing. Pushed empty col for better spacing.
…ther on demand or automatically. (DeviaVir#785)
* Added Binance to Zenbot * small change in the engine to make binance cancel order work * removed the node-binance-api * fixed feedback for pull request.
* Added Binance to Zenbot * small change in the engine to make binance cancel order work * removed the node-binance-api * fixed feedback for pull request. * Added API support to zenbot * Added output handler in the lib * Fixed feedback added readme
…#794) * Fix automatic products generation. Fix commi 757 that overwrote minimun increment for kraken products * Catching more recoverable connection errors in Kraken * Fix indenting -Issue DeviaVir#800 : Fix QueryOrders price * Fix indenting
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.