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

feat: refactor ICQ module documentation [NTRN-349] #646

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

sotnikov-s
Copy link
Contributor

@sotnikov-s sotnikov-s commented Jul 31, 2024

task

https://hadronlabs.atlassian.net/browse/NTRN-349

this PR

  • covers all .proto and sudo definitions of the interchainqueries module with comments aiming to make use of the descriptions via links to code based documentation (e.g. QueryRegisteredQueriesRequest)

related PRs

notes

  • Some types have been renamed to achieve consistent naming pattern. This should not contain any breaking changes to the blockchain interface because names of the request/response/etc types do not affect the interface in any way. Please correct me if I'm wrong.

@sotnikov-s sotnikov-s marked this pull request as draft July 31, 2024 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant