-
Notifications
You must be signed in to change notification settings - Fork 3.9k
docs: v0.53.x upgrade guide #24245
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
docs: v0.53.x upgrade guide #24245
Conversation
Required wiring: | ||
- Module Account Permissions | ||
- protocolpooltypes.ModuleName (nil) |
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.
not really sure how verbose we want to get here. i initially put some code stubs for every piece of wiring, but it felt super overwhelming. considering there are 2 configurations (DI, non-DI).
not sure if this doc is meant to be a long winded document tutorial, or just a quick reference for whats changing and whats new.
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.
I think we should have this guide as is (pretty light) and then have literal step-by-step guides for DI and non-DI users
Required wiring: | ||
- Module Account Permissions | ||
- protocolpooltypes.ModuleName (nil) |
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.
I think we should have this guide as is (pretty light) and then have literal step-by-step guides for DI and non-DI users
Co-authored-by: Alex | Interchain Labs <[email protected]>
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.
This lgtm - then we can follow up with step by step guides
* main: (2959 commits) docs: upgrade guide (#24333) update test: fix output formatting, typo corrections, and test case expectations in dec tests (#24325) fix: use the gogoproto merge registry as a file resolver instead of the interface registry (#24330) fix(x/auth): set from flag in multi-sign command (backport #19239) (backport #24265) (#24331) docs: v0.53.x upgrade guide (#24245) chore: update the core and collections deps (#24324) build(deps): Bump the npm_and_yarn group across 1 directory with 8 updates (#24328) chore: update deps to silence dependatbot (#24327) docs(math): backport decimal-handling.md (#24312) build(deps): Bump the npm_and_yarn group across 1 directory with 16 updates (#24323) build(deps): Bump actions/setup-go from 4 to 5 (#24272) build(deps): Bump actions/cache from 3 to 4 (#24268) build(deps): Bump github/codeql-action from 2 to 3 (#24270) build(deps): Bump nanoid from 3.3.6 to 3.3.11 in /docs (#24294) build(deps): Bump serialize-javascript from 6.0.0 to 6.0.2 in /docs (#24300) build(deps): Bump @babel/helpers from 7.19.4 to 7.27.0 in /docs (#24309) build(deps): Bump actions/setup-node from 3 to 4 (#24282) build(deps): Bump @babel/runtime-corejs3 from 7.19.4 to 7.27.0 in /docs (#24308) build(deps): Bump cross-spawn from 7.0.3 to 7.0.6 in /docs (#24307) ...
Description
Closes: #24078
Upgrade guide for v0.50.x -> v0.53.x.
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...
!
in the type prefix if API or client breaking changeCHANGELOG.md
Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
Please see Pull Request Reviewer section in the contributing guide for more information on how to review a pull request.
I have...