-
Notifications
You must be signed in to change notification settings - Fork 586
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
Document ibc app expectations and recommendations #5506
Conversation
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.
lgtm. Maybe we should mention that OnChanUpgradeInit and OnChanUpgradeTry can also propose version themselves (fully or partially)?
good call, will add a note about it |
bea23c8
into
cian/issue#5466-update-documentation-on-upgradablemodule-interface-functions
* chore: merge main * chore: corrected commit * Document ibc app expectations and recommendations (#5506) * fix typo --------- Co-authored-by: Charly <charly@interchain.io> Co-authored-by: Carlos Rodriguez <carlos@interchain.io>
* chore: merge main * chore: corrected commit * Document ibc app expectations and recommendations (#5506) * fix typo --------- Co-authored-by: Charly <charly@interchain.io> Co-authored-by: Carlos Rodriguez <carlos@interchain.io> (cherry picked from commit 26fd1e0) Co-authored-by: Cian Hatton <cian@interchain.io>
Description
closes: #4277
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/
) or specification (x/<module>/spec/
).godoc
comments.Files changed
in the Github PR explorer.Codecov Report
in the comment section below once CI passes.