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

Small fix for Cexio #299

Closed
wants to merge 4 commits into from
Closed

Small fix for Cexio #299

wants to merge 4 commits into from

Conversation

horsecoin
Copy link

Changes I made to get Cexio exchange to work. Please Mike take a look-if good pull, else fix!
Thanks!

Example error it was throwing before fixes:

TypeError: Cannot read property 'amount' of undefined

at Manager.getBalance (/home/nick/gekko2/core/portfolioManager.js:108:28)

at bound [as getBalance] (/home/nick/gekko2/node_modules/lodash/dist/lodash.js:729:21)

at act (/home/nick/gekko2/core/portfolioManager.js:130:26)

at bound (/home/nick/gekko2/node_modules/lodash/dist/lodash.js:729:21)

at /home/nick/gekko2/node_modules/async/lib/async.js:232:13

at /home/nick/gekko2/node_modules/async/lib/async.js:142:25

at /home/nick/gekko2/node_modules/async/lib/async.js:229:17

at /home/nick/gekko2/node_modules/async/lib/async.js:556:34

at set (/home/nick/gekko2/core/portfolioManager.js:78:7)

at bound (/home/nick/gekko2/node_modules/lodash/dist/lodash.js:729:21)

@horsecoin
Copy link
Author

had issues- back to drawing board

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.

None yet

1 participant