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.

Sign up for GitHub

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

Jump to bottom

New Bonds Info List Query #225

Merged
merged 4 commits into from
Feb 22, 2021
Merged

New Bonds Info List Query #225

merged 4 commits into from
Feb 22, 2021

Conversation

migueldingli1997
Copy link
Collaborator

@migueldingli1997 migueldingli1997 commented Feb 17, 2021 •
edited
Loading

Main changes:

  • Added new detailed bonds list query (with spot price, supply, and reserve) for a concise snapshot of some of the bond state, especially useful for data aggregators

Other

  • Fixed pruning endpoints (everything -> nothing) to keep all data
  • Fixed REST queries to use keeper types instead of hard-coded strings in QueryWithData calls

@migueldingli1997 migueldingli1997 changed the title WIP: New Bonds Info List Query New Bonds Info List Query Feb 21, 2021
@migueldingli1997 migueldingli1997 merged commit 32b93dc into master Feb 22, 2021
@migueldingli1997 migueldingli1997 deleted the miguel/all-bonds-info-endpoint branch February 22, 2021 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stefaniatadama stefaniatadama stefaniatadama approved these changes

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
@migueldingli1997 @stefaniatadama