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

Add GeneralAdmin as Origin of multiple extrinsics (Moonbase/Moonriver) #2044

Merged
merged 1 commit into from
Jan 17, 2023

Conversation

crystalin
Copy link
Collaborator

Adds the GeneralAdmin (OpenGov Track origin) as the Origin for those Extrinsics in Moonbase/Moonriver
(Council Collective for actual governance is kept for satefy purposes until OpenGov is final)

Breaking

  • Those extrinsics are now controllable by the GeneralAdmin origin (associated with the GeneralAdmin OpenGov track

Extrinsics

origin parachainStaking.MonetaryGovernanceOrigin:

  • parachainStaking.setStakingExpectations
  • parachainStaking.setInflation
  • parachainStaking.setParachainBondAccount
  • parachainStaking.setParachainBondReservePercent

origin identity.RegistrarOrigin:

  • identity.addRegistrar

origin identity.ForceOrigin:

  • identity.killIdentity

origin assets.ForceOrigin:

  • assets.forceCreate
  • assets.destroy
  • assets.forceSetMetadata
  • assets.forceClearMetadata
  • assets.forceAssetStatus
  • assets.forceCancelApproval

origin assetManager.ForeignAssetModifierOrigin:

  • assetManager.registerForeignAsset
  • assetManager.setAssetUnitsPerSecond
  • assetManager.changeExistingAssetType
  • assetManager.removeSupportedAsset
  • assetManager.removeExistingAssetType
  • assetManager.destroyForeignAsset

origin assetManager.LocalAssetModifierOrigin:

  • assetManager.registerLocalAsset
  • assetManager.destroyLocalAsset

origin xcmTransactor.DerivativeAddressRegistrationOrigin:

  • xcmTransactor.register
  • xcmTransactor.deregister
  • xcmTransactor.setTransactInfo
  • xcmTransactor.removeTransactInfo
  • xcmTransactor.setFeePerSecond
  • xcmTransactor.removeFeePerSecond

origin localAssets.ForceOrigin:

  • localAssets.forceCreate
  • localAssets.destroy
  • localAssets.forceSetMetadata
  • localAssets.forceClearMetadata
  • localAssets.forceAssetStatus
  • localAssets.forceCancelApproval

origin moonbeamOrbiters.AddCollatorOrigin:

  • moonbeamOrbiters.addCollator

origin moonbeamOrbiters.DelCollatorOrigin:

  • moonbeamOrbiters.removeCollator

@crystalin crystalin added A0-pleasereview Pull request needs code review. B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes D9-needsaudit👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited breaking Needs to be mentioned in breaking changes labels Jan 17, 2023
@crystalin crystalin merged commit 51adcd1 into master Jan 17, 2023
@crystalin crystalin deleted the crystalin-general-admin-origins branch January 17, 2023 14:32
@librelois librelois mentioned this pull request Jan 17, 2023
19 tasks
@crystalin crystalin changed the title Adds GeneralAdmin as Origin of multiple extrinsics (Moonbase/Moonriver) Add GeneralAdmin as Origin of multiple extrinsics (Moonbase/Moonriver) Jan 20, 2023
@notlesh notlesh added D1-audited👍 PR contains changes to fund-managing logic that has been properly reviewed and externally audited and removed D9-needsaudit👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited labels Feb 14, 2023
imstar15 pushed a commit to AvaProtocol/moonbeam that referenced this pull request May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A0-pleasereview Pull request needs code review. B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes breaking Needs to be mentioned in breaking changes D1-audited👍 PR contains changes to fund-managing logic that has been properly reviewed and externally audited
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants