-
Notifications
You must be signed in to change notification settings - Fork 371
Companion: Accept Kusama StakingAdmin origin #1865
Conversation
joepetrowski
left a comment
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 think we should probably make a new BodyId::TechnicalFellowship to avoid granting Fellowship power to the old Technical Committee.
|
|
||
| parameter_types! { | ||
| // Fellows pluralistic body. | ||
| pub const FellowsBodyId: BodyId = BodyId::Technical; |
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 think that as long as the Tech Committee and Fellowship coexist, the Tech Comm would be able to control this origin.
https://github.com/paritytech/polkadot/pull/6273/files#r1049712864
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.
Right, but this should be valid?
Tech Committee has a root permissions.
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, they do not. Only fast tracking permission.
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.
Double checked, Technical xcm body not used/mapped right now in kusama or polkadot (only docs tells as its supposed to be used as Tech Comm), it's safe to start using it as Fellows.
|
bot rebase |
|
Rebased |
|
bot rebase |
|
Rebased |
|
bot merge |
* Companion: Accept Kusamsa StakinAdmin origin * Fellows origin for xcmp queue controller * plurality bodies to ensure origin * include root into controller ensure origin * fix * use xcm bodies * rename to more frequent "defense" Co-authored-by: parity-processbot <>
* Companion: Accept Kusamsa StakinAdmin origin * Fellows origin for xcmp queue controller * plurality bodies to ensure origin * include root into controller ensure origin * fix * use xcm bodies * rename to more frequent "defense" Co-authored-by: parity-processbot <>
|
This pull request has been mentioned on Polkadot Forum. There might be relevant details there: |
4c4a7eae1 Small stuff from Cumulus integration (#1865) git-subtree-dir: bridges git-subtree-split: 4c4a7eae1503aa63a84fb65d56d67599d362d645
Companion for paritytech/polkadot#6273