-
Notifications
You must be signed in to change notification settings - Fork 26
Update RPC Tendermint to 38 #273
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
Update RPC Tendermint to 38 #273
Conversation
@@ -11,6 +11,7 @@ use prost::Message; | |||
|
|||
/// Querier for the Tendermint node. | |||
/// Supports queries for block and tx information | |||
/// @TODO: all tendermint queries should use the tendermint-rpc explicitly instead of hitting the tendermint node with the typed queries. |
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.
Outstanding issue
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.
what do you mean ? The tendermint rpc library does not expose those types, so we can't really do that can we ?
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.
Only when it does expose those types - I think most of them are covered in this PR but there could still be some outstanding ones.
Closing because we are not planning on adding RPC support to cw-orch soon. |
This changes updates the version of
cosmrs
for RPC support in orchestrator. It also fixes some minor bugs when interacting withwasmd
v0.50-rc-2.