-
Notifications
You must be signed in to change notification settings - Fork 20
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
feat: Sunset gov v1 #670
feat: Sunset gov v1 #670
Conversation
Crate versions that have been updated:
Runtime version has been increased. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fmt & bump versions & LGMT
@@ -170,13 +171,9 @@ construct_runtime!( | |||
Treasury: pallet_treasury = 4, | |||
Utility: pallet_utility = 5, | |||
//NOTE: 6 - is used by Scheduler which must be after cumulus_pallet_parachain_system | |||
Democracy: pallet_democracy exclude_parts { Config } = 7, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should probably retire those pallet id's of removed pallets
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wdym by retire the pallet IDs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not use them in future
@@ -171,12 +172,9 @@ construct_runtime!( | |||
Utility: pallet_utility = 5, | |||
//NOTE: 6 - is used by Scheduler which must be after cumulus_pallet_parachain_system | |||
Democracy: pallet_democracy exclude_parts { Config } = 7, | |||
Elections: pallet_elections_phragmen = 8, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should check the storage after we remove pallets, I think we'll need to clear it either by ref or by migration.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I see later we do migration
@@ -19,29 +19,12 @@ use crate::governance::TechnicalCollective; | |||
use crate::*; | |||
|
|||
use primitives::constants::{ | |||
currency::{CENTS, DOLLARS, UNITS}, | |||
currency::DOLLARS, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why remove this and then use imports directly?
This PR sunsets gov v1 and the Council, more specifically:
elections-phragmen
,tips
,collectives/council
pallet-democracy
is left in the runtime for now, to be cleaned up later once the existing votes have expired (another max ~3 months)