We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
--height
a) gaiad q liquidity pools --height 1 produces the same output than gaiad q liquidity pools --height 10638712 than gaiad q liquidity pools
gaiad q liquidity pools --height 1
gaiad q liquidity pools --height 10638712
gaiad q liquidity pools
I think the problem is here: https://github.com/Gravity-Devs/liquidity/blob/803b1c5c0d483f02e0c54695ae3ee6a5f2068bd0/x/liquidity/client/cli/query.go#LL106C2-L109
Maybe client.GetClientTxContext(cmd) should be replaced by client.GetClientQueryContext(cmd)
client.GetClientTxContext(cmd)
client.GetClientQueryContext(cmd)
b) The same counts with LCD queries, but the previous fix doesn't works there.
You can see it at gaiad v9.x
Indicated at the summary.
The text was updated successfully, but these errors were encountered:
fix: use GetClientQueryContext for --height flag on cli #28
f74697a
No branches or pull requests
Summary of Bug
a)
gaiad q liquidity pools --height 1
produces the same output thangaiad q liquidity pools --height 10638712
thangaiad q liquidity pools
I think the problem is here:
https://github.com/Gravity-Devs/liquidity/blob/803b1c5c0d483f02e0c54695ae3ee6a5f2068bd0/x/liquidity/client/cli/query.go#LL106C2-L109
Maybe
client.GetClientTxContext(cmd)
should be replaced byclient.GetClientQueryContext(cmd)
b) The same counts with LCD queries, but the previous fix doesn't works there.
Version
You can see it at gaiad v9.x
Steps to Reproduce
Indicated at the summary.
For Admin Use
The text was updated successfully, but these errors were encountered: