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

Add query rpc for interchain account address to controller submodule #2192

Closed
6 tasks
damiannolan opened this issue Sep 6, 2022 · 0 comments · Fixed by #2193
Closed
6 tasks

Add query rpc for interchain account address to controller submodule #2192

damiannolan opened this issue Sep 6, 2022 · 0 comments · Fixed by #2193
Assignees
Milestone

Comments

@damiannolan
Copy link
Member

Summary

A new query rpc method should be added to the controller submodule to allow end users the ability to inspect the interchain account address generated on the host chain.

  • Add the rpc endpoint to the controller submodule Query service
  • Add the implementation to the controller submodule Keeper
  • Add a CLI query to the controller submodule queryCmd
service Query {
  // InterchainAccount returns the interchain account address for a given owner address on a given connection
  rpc InterchainAccount(QueryInterchainAccountRequest) returns (QueryInterchainAccountResponse) {}

...
}

// QueryInterchainAccountRequest is the request type for the Query/InterchainAccount RPC method.
message QueryInterchainAccountRequest {
  string owner         = 1;
  string connection_id = 2 [(gogoproto.moretags) = "yaml:\"connection_id\""];
}

// QueryInterchainAccountResponse the response type for the Query/InterchainAccount RPC method.
message QueryInterchainAccountResponse {
  string address = 1;
}

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged/assigned
@damiannolan damiannolan added this to the v5.1.0 milestone Sep 6, 2022
@damiannolan damiannolan self-assigned this Sep 6, 2022
@damiannolan damiannolan moved this to In review in ibc-go Sep 6, 2022
@damiannolan damiannolan moved this from In review to In progress in ibc-go Sep 6, 2022
@colin-axner colin-axner moved this from In progress to In review in ibc-go Sep 7, 2022
Repository owner moved this from In review to Done in ibc-go Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant