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.

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

Enable Nomination Pool Participation in Governance in Kusama #540

Merged

Conversation

Ank4n
Copy link
Contributor

@Ank4n Ank4n commented Jan 12, 2025

Relates to #455.

The Polkadot changes were implemented in this PR but were reverted for Kusama. Some patches to 2409 were required before these changes can be applied to Kusama.

@Ank4n Ank4n mentioned this pull request Jan 13, 2025
6 tasks
@@ -1859,6 +1867,11 @@ pub mod migrations {
parachains_inclusion::migration::MigrateToV1<Runtime>,
parachains_on_demand::migration::MigrateV0ToV1<Runtime>,
restore_corrupted_ledgers::Migrate<Runtime>,
// Migrate NominationPools to `DelegateStake` adapter. This is an unversioned upgrade.
pallet_nomination_pools::migration::unversioned::DelegationStakeMigration<
Copy link
Contributor

Choose a reason for hiding this comment

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

Fine if this is called multiple times?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah it ignores pool if already migrated.

@kianenigma
Copy link
Contributor

Let's hype this up a bit more by renaming it to "Enable Nomination Pool Participation in Governance in Kusama"? 🙈

@Ank4n Ank4n changed the title Enable delegate stake in kusama Enable Nomination Pool Participation in Governance in Kusama Jan 13, 2025
@Ank4n
Copy link
Contributor Author

Ank4n commented Jan 13, 2025

/merge

@fellowship-merge-bot fellowship-merge-bot bot merged commit 59802a1 into polkadot-fellows:main Jan 13, 2025
54 of 56 checks passed
@fellowship-merge-bot
Copy link
Contributor

Enabled auto-merge in Pull Request

Available commands
  • /merge: Enables auto-merge for Pull Request
  • /merge cancel: Cancels auto-merge for Pull Request
  • /merge help: Shows this menu

For more information see the documentation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants