Skip to content

Commit

Permalink
doc: adding migration doc info for nil legacy subspace (#4816)
Browse files Browse the repository at this point in the history
  • Loading branch information
damiannolan authored Oct 5, 2023
1 parent a948e34 commit 2fb6171
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions docs/docs/05-migrations/11-v7-to-v8.md
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

0 comments on commit 2fb6171

Please sign in to comment.