-
Notifications
You must be signed in to change notification settings - Fork 101
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
Fast Fellowship Promote Tracks #356
Conversation
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
const FAST_PROMOTE_MAX_DECIDING: u32 = 10; | ||
const FAST_PROMOTE_DECISION_DEPOSIT: Balance = 5 * DOLLARS; | ||
const FAST_PROMOTE_PREPARE_PERIOD: BlockNumber = 0; | ||
const FAST_PROMOTE_DECISION_PERIOD: BlockNumber = 7 * DAYS; |
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.
Dunno if this is too short? The origin is already "faster" since it can bypass the promotion period.
On the other hand, with the high turnout it will rather error on the side of rejecting so maybe its fine.
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.
The main issue is having to wait for years to get to a correct level.
If this can bypass the promotion min period, I don't see the need to have it faster than the "normal" 30 days, I don't expect members to find 30 days too slow..
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.
The main issue is having to wait for years to get to a correct level.
Yeah that needs to be fixed by making a call in Core Fellowship that will respect this origin.
Dunno if this is too short?
I think it is OK, could also go in the middle with something like 14 days.
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.
No strong opinions, so changed to 30 days after internal discussion.
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
/merge |
Enabled Available commands
For more information see the documentation |
Add fast promotion tracks with 7 days (instead of 30 days) voting period.
The tracks have a 50% turnout and 66% approval criteria. To be used in combination with paritytech/polkadot-sdk#4778.
It could be that the 50% turnout is too much for tracks I and II, but for III there are not so many fellows so we can expect them to all vote IMHO.