Skip to content

Backport 4229 Into v1.7.0#4288

Merged
EgorPopelyaev merged 4 commits into
release-crates-io-v1.7.0from
joe-backport-4229
Apr 25, 2024
Merged

Backport 4229 Into v1.7.0#4288
EgorPopelyaev merged 4 commits into
release-crates-io-v1.7.0from
joe-backport-4229

Conversation

@joepetrowski

Copy link
Copy Markdown
Contributor

Backport of #4229 into the v1.7.0 release in order to support dependent runtimes. This version of the SDK doesn't have UncheckedOnRuntimeUpgrade, so I do it the "old way".

@joepetrowski joepetrowski added the A3-backport Pull request is already reviewed well in another branch. label Apr 25, 2024
@paritytech-cicd-pr

Copy link
Copy Markdown

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-clippy
Logs: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/6056395

@EgorPopelyaev EgorPopelyaev merged commit 06d0d22 into release-crates-io-v1.7.0 Apr 25, 2024
@EgorPopelyaev EgorPopelyaev deleted the joe-backport-4229 branch April 25, 2024 11:46
Comment on lines +43 to +44
pub struct MigrationToV2<T>(sp_std::marker::PhantomData<T>);
impl<T: Config + pallet_balances::Config> OnRuntimeUpgrade for MigrationToV2<T> {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are we not doing VersionedMigration + UncheckedMigrationToV2 as in #4229 ?

EgorPopelyaev pushed a commit that referenced this pull request Apr 26, 2024
Backport of #4229 into the v1.7.0 release in order to support dependent
runtimes. This version of the SDK doesn't have
`UncheckedOnRuntimeUpgrade`, so I do it the "old way".
EgorPopelyaev pushed a commit that referenced this pull request Apr 26, 2024
Backport of #4229 into the v1.7.0 release in order to support dependent
runtimes. This version of the SDK doesn't have
`UncheckedOnRuntimeUpgrade`, so I do it the "old way".
EgorPopelyaev pushed a commit that referenced this pull request Apr 26, 2024
Backport of #4229 into the v1.7.0 release in order to support dependent
runtimes. This version of the SDK doesn't have
`UncheckedOnRuntimeUpgrade`, so I do it the "old way".
EgorPopelyaev added a commit that referenced this pull request Apr 26, 2024
Backport of #4229 into the v1.7.0 release in order to support dependent
runtimes. This version of the SDK doesn't have
`UncheckedOnRuntimeUpgrade`, so I do it the "old way".
EgorPopelyaev pushed a commit that referenced this pull request Apr 26, 2024
Backport of #4229 into the v1.7.0 release in order to support dependent
runtimes. This version of the SDK doesn't have
`UncheckedOnRuntimeUpgrade`, so I do it the "old way".
EgorPopelyaev pushed a commit that referenced this pull request Apr 26, 2024
Backport of #4229 into the v1.7.0 release in order to support dependent
runtimes. This version of the SDK doesn't have
`UncheckedOnRuntimeUpgrade`, so I do it the "old way".
@Polkadot-Forum

Copy link
Copy Markdown

This pull request has been mentioned on Polkadot Forum. There might be relevant details there:

https://forum.polkadot.network/t/paritytech-update-for-april-2024/7646/8

bkchr pushed a commit that referenced this pull request Jun 6, 2024
Backport of #4229 into the v1.7.0 release in order to support dependent
runtimes. This version of the SDK doesn't have
`UncheckedOnRuntimeUpgrade`, so I do it the "old way".
bkchr pushed a commit that referenced this pull request Jun 6, 2024
Backport of #4229 into the v1.7.0 release in order to support dependent
runtimes. This version of the SDK doesn't have
`UncheckedOnRuntimeUpgrade`, so I do it the "old way".
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A3-backport Pull request is already reviewed well in another branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants