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

cli (vpool): query asset prices #547

Merged

Conversation

kai0x
Copy link
Contributor

@kai0x kai0x commented Jun 6, 2022

No description provided.

@kai0x kai0x linked an issue Jun 6, 2022 that may be closed by this pull request
kai0x added 2 commits June 6, 2022 01:40
needs more investigation on test setup for this one
@kai0x kai0x changed the title test(perp): query asset prices cli (vpool): query asset prices Jun 6, 2022
@kai0x kai0x marked this pull request as ready for review June 6, 2022 20:32
@kai0x kai0x requested a review from a team as a code owner June 6, 2022 20:32
proto/vpool/v1/query.proto Outdated Show resolved Hide resolved
x/vpool/keeper/prices.go Outdated Show resolved Hide resolved
x/vpool/keeper/prices.go Outdated Show resolved Hide resolved
x/vpool/keeper/query_server.go Outdated Show resolved Hide resolved
kai0x and others added 6 commits June 7, 2022 21:18
Co-authored-by: Walter White <101130700+NibiruHeisenberg@users.noreply.github.com>
Co-authored-by: Walter White <101130700+NibiruHeisenberg@users.noreply.github.com>
@kai0x
Copy link
Contributor Author

kai0x commented Jun 9, 2022

addressed all the comments 👍 lmk if any other changes would be helpful

proto/vpool/v1/query.proto Outdated Show resolved Hide resolved
proto/vpool/v1/query.proto Outdated Show resolved Hide resolved
proto/vpool/v1/query.proto Outdated Show resolved Hide resolved
x/testutil/cli/query.go Outdated Show resolved Hide resolved
x/testutil/cli/query.go Outdated Show resolved Hide resolved
kai0x and others added 2 commits June 9, 2022 22:18
Co-authored-by: Walter White <101130700+NibiruHeisenberg@users.noreply.github.com>
Co-authored-by: Walter White <101130700+NibiruHeisenberg@users.noreply.github.com>
kai0x and others added 2 commits June 9, 2022 22:18
Co-authored-by: Walter White <101130700+NibiruHeisenberg@users.noreply.github.com>
@NibiruHeisenberg NibiruHeisenberg merged commit fde1694 into master Jun 10, 2022
@NibiruHeisenberg NibiruHeisenberg deleted the 546-perp-create-a-getbaseassetprice-cli-command branch June 10, 2022 15:45
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.

[perp] create a GetBaseAssetPrice cli command
4 participants