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.

Sign up for GitHub

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

doc: adding migration doc info for nil legacy subspace #4816

Merged
merged 5 commits into from
Oct 5, 2023
Merged

doc: adding migration doc info for nil legacy subspace #4816

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
8435e4b
chore: adding migration doc info for nil legacy subspace
damiannolan Oct 4, 2023
9594a17
fix: lint
damiannolan Oct 4, 2023
b9e18f3
fix: more indentation because
damiannolan Oct 4, 2023
b08653e
Merge branch 'main' into damian/update-migration-doc
damiannolan Oct 4, 2023
afc5e0e
Merge branch 'main' into damian/update-migration-doc
Oct 4, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions docs/docs/05-migrations/11-v7-to-v8.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,15 @@ There are four sections based on the four potential user groups of this document

- [Migrating from v7 to v8](#migrating-from-v7-to-v8)
- [Chains](#chains)
- [Cosmos SDK v0.50 upgrade](#cosmos-sdk-v050-upgrade)
- [Authority](#authority)
- [Testing package](#testing-package)
- [Params migration](#params-migration)
- [Governance V1 migration](#governance-v1-migration)
- [Transfer migration](#transfer-migration)
- [IBC Apps](#ibc-apps)
- [ICS20 - Transfer](#ics20---transfer)
- [ICS27 - Interchain Accounts](#ics27---interchain-accounts)
- [Relayers](#relayers)
- [IBC Light Clients](#ibc-light-clients)

Expand Down Expand Up @@ -138,6 +146,8 @@ Each module has a corresponding `MsgUpdateParams` message with a `Params` which

Legacy params subspaces must still be initialised in app.go in order to successfully migrate from x/params to the new self-contained approach. See reference: <https://github.com/cosmos/ibc-go/blob/main/testing/simapp/app.go#L1001-L1006>

For new chains which do not rely on migration of parameters from `x/params`, an expected interface has been added for each module. This allows chain developers to provide `nil` as the `legacySubspace` argument to `NewKeeper` functions.

### Governance V1 migration

Proposals have been migrated to [gov v1 messages](https://docs.cosmos.network/v0.50/modules/gov#messages) ref: [#4620](https://github.com/cosmos/ibc-go/pull/4620). The proposal `ClientUpdateProposal` has been deprecated and [`MsgRecoverClient`](https://github.com/cosmos/ibc-go/blob/v8.0.0-beta.0/proto/ibc/core/client/v1/tx.proto#L121-L134) should be used instead. Likewise, the proposal `UpgradeProposal` has been deprecated and [`MsgIBCSoftwareUpgrade`](https://github.com/cosmos/ibc-go/blob/v8.0.0-beta.0/proto/ibc/core/client/v1/tx.proto#L139-L154) should be used instead. Both proposals will be removed in the next major release.
Expand Down
Loading