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 mto_id to ticker data #2237

Merged
merged 2 commits into from
Aug 2, 2020
Merged

Add mto_id to ticker data #2237

merged 2 commits into from
Aug 2, 2020

Conversation

syalon
Copy link
Member

@syalon syalon commented Jul 28, 2020

User Story

In some cases, we need to directly use the market_ticker_object object to obtain ticker data in batches. The current ticker api can only obtain a single trading pair. It is not necessary to add a new api, so we simply add mto_id to the current ticker data.

Updated by @abitmore: related to issue #1937.

Impacts

Describe which portion(s) of BitShares Core may be impacted by your request. Please tick at least one box.

  • API (the application programming interface)
  • UI (the user interface)
  • Build (the build process or something prior to compiled code)
  • CLI (the command line wallet)
  • Deployment (the deployment process after building such as Docker, Travis, etc.)
  • DEX (the Decentralized EXchange, market engine, etc.)
  • P2P (the peer-to-peer network for transaction/block propagation)
  • Performance (system or user efficiency, etc.)
  • Protocol (the blockchain logic, consensus, validation, etc.)
  • Security (the security of system or user data, etc.)
  • UX (the User Experience)
  • Other (please add below)

@abitmore
Copy link
Member

Please elaborate (motivation and rationale).

@abitmore abitmore added this to the Future Feature Release milestone Jul 28, 2020
Copy link
Member

@abitmore abitmore left a comment

Choose a reason for hiding this comment

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

Thanks. I'll merge this as a temporary solution.

@abitmore abitmore merged commit 6cee0a1 into bitshares:develop Aug 2, 2020
@abitmore abitmore mentioned this pull request Aug 2, 2020
17 tasks
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