Skip to content
This repository has been archived by the owner on Jun 24, 2024. It is now read-only.

feat: implement IBC query endpoints #9

Merged
merged 6 commits into from
Oct 7, 2023
Merged

Conversation

Farhad-Shabani
Copy link
Member

@Farhad-Shabani Farhad-Shabani commented Oct 6, 2023

Closes: #3
Closes: #8
Pertaining PR on the Sovereign SDK front: informalsystems/sovereign-sdk#16

Description

As a result of improvements made in ibc-rs through this PR, we are now able to fully implement IBC query interface. Within this PR We've got the essential IBC query endpoints up and running, all set to go with the demo rollup (see the instruction here)

Later

  • For a complete set of IBC query endpoints, we can tap into supplementary query methods by implementing QueryContext. Kept it as a separate issue since it's not as high-priority right now.
  • As for the get_proof method, I've opened this issue.

PR author checklist

  • Linked to GitHub issue.
  • Added tests.
  • Updated code comments and documentation (e.g., docs/).
  • Tagged one reviewer who will be the one responsible for shepherding this PR.

@Farhad-Shabani Farhad-Shabani marked this pull request as ready for review October 6, 2023 17:15
@Farhad-Shabani Farhad-Shabani merged commit 5c583d8 into main Oct 7, 2023
7 checks passed
@Farhad-Shabani Farhad-Shabani deleted the farhad/rpc-impl branch October 7, 2023 01:07
@Farhad-Shabani Farhad-Shabani self-assigned this Oct 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: ✅ Done
1 participant