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

query ConsensusState given client id and height #129

Closed
ebuchman opened this issue Jul 15, 2020 · 2 comments
Closed

query ConsensusState given client id and height #129

ebuchman opened this issue Jul 15, 2020 · 2 comments

Comments

@ebuchman
Copy link
Member

Given a client id and height, query the ConsensusState.

This is similar to cosmos/ibc-proto-rs#21 in that ConsensusState is an interface with concrete implementations depending on the client type (eg. for Tendermint). So it has Tendermint specific data structures.

This type is simpler than the ClientState, but it also doesn't have a proto file yet. It's defined as a Go struct.. Since it contains the validator set, it will depend on Amino for now ...

ABCIQuery.Path = store/ibc/key
ABCIQuery.Data = clients/{identifier}/consensusState/{height}
@ancazamfir ancazamfir mentioned this issue Jul 15, 2020
Closed
15 tasks
@ebuchman
Copy link
Member Author

This is blocked on the SDK upgrading to use the tendermint v0.34 release (currently a Release Candidate), but that should probably happen shortly.

@ancazamfir
Copy link
Collaborator

ancazamfir commented Oct 15, 2020 •
edited
Loading

Finished after all the protobuf updates, tm releases, informalsystems/hermes#235, informalsystems/hermes#292, #106.

@hu55a1n1 hu55a1n1 transferred this issue from informalsystems/hermes Sep 29, 2022
shuoer86 pushed a commit to shuoer86/ibc-rs that referenced this issue Nov 4, 2023
@damiannolan @colin-axner
fix: protobuf formatting using clang-format (cosmos#129)
9919ce9 
* fix: use /go suffix in go package option

* adding .clang-format for protobuf formatting

Co-authored-by: Colin Axnér <25233464+colin-axner@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ebuchman @ancazamfir