Skip to content
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

Fix ticker models #7

Closed
wants to merge 1 commit into from
Closed

Conversation

ddeath
Copy link

@ddeath ddeath commented Nov 14, 2018

Fix ticker models.

Why the test does not run against real api?

@ddeath
Copy link
Author

ddeath commented Nov 15, 2018

I see that you removed that a day ago. My question is why if the node library is not working at the moment with the removed symbol.

@fieldfoxWim
Copy link

Just saw your pull request after I created an issue for it. #10

Culprit commit 2b46b6c
by @leehillz

@leehillz please advice

@f3rno
Copy link
Contributor

f3rno commented Dec 4, 2018

I cannot reproduce on latest master of this repo, nor with the latest released version. Can you confirm it is still an issue if you run npm update bfx-api-node-models in your project repos?

@f3rno
Copy link
Contributor

f3rno commented Feb 26, 2020

@JacobPlaster can be closed I think

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants