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

Add market API. close #99 #100

Merged
merged 1 commit into from
Mar 14, 2017
Merged

Add market API. close #99 #100

merged 1 commit into from
Mar 14, 2017

Conversation

jcalfee
Copy link
Contributor

@jcalfee jcalfee commented Mar 13, 2017

No description provided.

@bonustrack
Copy link
Contributor

bonustrack commented Mar 14, 2017

@jcalfee What's the reason of getMarketOrderBook, getOrderBook wouldn't work?

@jcalfee
Copy link
Contributor Author

jcalfee commented Mar 14, 2017

The purpose is to resolve a conflicting name. getOrderBook is part of the database_api too.. Fyi: it is not intuitive, but the Market pages in Steemit uses getOrderBook from the database_api while making use of several other market_history_api methods ..

@jcalfee jcalfee merged commit 7cee867 into master Mar 14, 2017
@bonustrack bonustrack deleted the 99-market-api branch April 12, 2017 11:51
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.

2 participants