-
Notifications
You must be signed in to change notification settings - Fork 9
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
feat: add oracle exchange rate query #45
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #45 +/- ##
==========================================
+ Coverage 79.65% 81.33% +1.67%
==========================================
Files 41 35 -6
Lines 3161 3434 +273
==========================================
+ Hits 2518 2793 +275
+ Misses 643 641 -2
☔ View full report in Codecov by Sentry. |
…ather than a struct
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Contract version in c3476da has passing e2e tests:
* feat: add oracle exchange rate query * fix: fix output of query type * chore: update binaries * fix: make exchange rate a sdk.dec * feat: request exchange rate with multiple pairs * fix: update readme * wip * feat(bindings-perp): fix oracle query and use simple hashmap for it rather than a struct --------- Co-authored-by: Unique-Divine <realuniquedivine@gmail.com>
This PR adds the exchange Rate query to the binding perp contract. We might want to rename binding perp at some point to encompass most of our interactions between wasm and the chain but since ExchangeRate is the only one we need outside of perp for now, i just included it here.