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

CEXIO error handling #783

Merged
merged 2 commits into from
Dec 4, 2017
Merged

CEXIO error handling #783

merged 2 commits into from
Dec 4, 2017

Conversation

@nedievas
Copy link
Contributor

@nedievas nedievas commented Dec 2, 2017

also fixes

/home/user/zenbot/lib/engine.js:291
              order.status = api_order.status
                                      ^

TypeError: Cannot read property 'status' of undefined

crash.

@nedievas
CEXIO error handling
08cdff0 
also fixes `/home/user/zenbot/lib/engine.js:291
              order.status = api_order.status` crash.
@nedievas
Update exchange.js
d3f558f
DeviaVir
DeviaVir approved these changes Dec 4, 2017
View reviewed changes
Copy link
Owner

@DeviaVir DeviaVir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM

@DeviaVir DeviaVir merged commit a12481b into DeviaVir:master Dec 4, 2017
@snyk-bot snyk-bot mentioned this pull request May 14, 2020
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@DeviaVir DeviaVir DeviaVir approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nedievas @DeviaVir