Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Migration for phragmen election locks migration #4710

Closed

Conversation

emostov
Copy link
Contributor

@emostov emostov commented Jan 13, 2022

Applies the migration from paritytech/substrate#10649 to polkadot and kusama. They current plan is to apply this migration in the runtime after paritytech/substrate#10646 is included. This way we can be sure the list of accounts to migrate will not grow because the bug allowing it will have been removed.

References the bug fixed in paritytech/substrate#10646

runtime/polkadot/src/lib.rs Outdated Show resolved Hide resolved
@emostov emostov added B7-runtimenoteworthy C1-low PR touches the given topic and has a low impact on builders. D9-needsaudit 👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited. E0-runtime_migration PR introduces code that might require downstream chains to run a runtime upgrade. labels Jan 14, 2022
@emostov emostov changed the title Companion for phragmen election locks fix Companion for phragmen election locks fix: #10649 Jan 14, 2022
@emostov emostov added the A4-companion A PR that needs a companion PR to merge in parallel for one of its downstream dependencies. label Jan 14, 2022
@emostov emostov added this to the v0.9.16 milestone Jan 14, 2022
@emostov emostov changed the title Companion for phragmen election locks fix: #10649 Companion for phragmen election locks migration: #10649 Jan 14, 2022
@emostov emostov modified the milestones: v0.9.16, v0.9.17 Jan 18, 2022
@emostov emostov removed the A4-companion A PR that needs a companion PR to merge in parallel for one of its downstream dependencies. label Jan 18, 2022
@emostov emostov changed the title Companion for phragmen election locks migration: #10649 Migration for phragmen election locks migration Jan 18, 2022
@emostov
Copy link
Contributor Author

emostov commented Jan 18, 2022

Closing until appropriate merge

@emostov emostov closed this Jan 18, 2022
@shawntabrizi shawntabrizi removed the D9-needsaudit 👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited. label Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. E0-runtime_migration PR introduces code that might require downstream chains to run a runtime upgrade.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants