-
Notifications
You must be signed in to change notification settings - Fork 68
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!: disable slashing #184
Conversation
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.
just ooc, is the equivocation fix somehow related to slashing?
runtime/src/lib.rs
Outdated
type SlashCancelOrigin = EnsureOneOf< | ||
AccountId, | ||
EnsureRoot<AccountId>, | ||
pallet_collective::EnsureProportionAtLeast<_3, _4, AccountId, CouncilCollective>, |
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.
as there will be only once council member, EnsureMember makes a bit more sense
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.
I have to fix babe/grandpa equivocations at first for slashing to start to work. I can make it in 2 issues/PRs if we want
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 so the slashing wasn't working in the first place, got it. Probably no need to split it at this point.
Bump runtime and spec_version pls |
Oops. I'll fix this. |
Description
Delay slashing to infinity so slashes can be canceled with sudo
Related Issue
#145