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

Astar Governance Integration #1384

Merged
merged 2 commits into from
Nov 21, 2024
Merged

Astar Governance Integration #1384

merged 2 commits into from
Nov 21, 2024

Conversation

Dinonard
Copy link
Member

@Dinonard Dinonard commented Nov 19, 2024

Governance V1 on Astar

Integration of governance related functionality into Astar runtime.
Mirrors the Shibuya solution

TODO

  • enable dispatch precompile calls

Params Overview

Collectives

  • collective motion duration is 5 days, giving enough time for the collective members to vote aye or nay

Democracy

  • launch period will last for 7 days, upgrading a new proposal into a referendum once per week
  • voting period will last for 7 days, allowing ample time for token holders to vote
  • enactment period will take 2 days in case referendum has passed
  • base locking period will be 9 days, to match with the dApp staking (conviction voting applies)
    • this shouldn't be a problem since active members will most likely be both staking & voting
  • minimum deposit for creating a democracy proposal will be 1000 ASTR

Treasury

  • 5% of the treasury request will be reserved as collateral, but reserved value will always be at least 100 ASTR and not more than 1000 ASTR
  • spend period for each treasury will be 7 days

Community Council

  • 2/3 agreement to register a new dApp into dApp staking
  • 4/5 agreement to unregister a dApp from dApp staking
  • 2/3 agreement to execute staking action (lock/unlock/stake/unstake)
  • 2/3 agreement to approve or reject a spending proposal

@Dinonard Dinonard added astar Related to Astar shibuya related to shibuya runtime This PR/Issue is related to the topic “runtime”. labels Nov 19, 2024
@Dinonard Dinonard self-assigned this Nov 19, 2024
@Dinonard Dinonard marked this pull request as ready for review November 19, 2024 15:07
Copy link
Contributor

@ipapandinas ipapandinas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - I hadn't reviewed the Shibuya implementation before, but you did a great job with this governance integration - kudos! 👏

ipapandinas
ipapandinas previously approved these changes Nov 19, 2024
Copy link

Code Coverage

Package Line Rate Branch Rate Health
pallets/xc-asset-config/src 50% 0%
chain-extensions/pallet-assets/src 56% 0%
chain-extensions/unified-accounts/src 0% 0%
pallets/astar-xcm-benchmarks/src/fungible 100% 0%
precompiles/dapp-staking/src/test 0% 0%
pallets/unified-accounts/src 86% 0%
pallets/dapp-staking/rpc/runtime-api/src 0% 0%
pallets/dapp-staking/src 83% 0%
pallets/inflation/src 89% 0%
pallets/collator-selection/src 92% 0%
pallets/vesting-mbm/src 89% 0%
pallets/dapp-staking/src/benchmarking 98% 0%
pallets/astar-xcm-benchmarks/src 86% 0%
precompiles/substrate-ecdsa/src 74% 0%
pallets/price-aggregator/src 82% 0%
precompiles/xcm/src 71% 0%
precompiles/unified-accounts/src 100% 0%
pallets/dynamic-evm-base-fee/src 89% 0%
chain-extensions/types/assets/src 0% 0%
pallets/astar-xcm-benchmarks/src/generic 100% 0%
pallets/collective-proxy/src 86% 0%
precompiles/dispatch-lockdrop/src 86% 0%
primitives/src 57% 0%
precompiles/assets-erc20/src 78% 0%
pallets/ethereum-checked/src 74% 0%
precompiles/dapp-staking/src 90% 0%
chain-extensions/types/unified-accounts/src 0% 0%
precompiles/sr25519/src 64% 0%
pallets/static-price-provider/src 85% 0%
pallets/dapp-staking/src/test 0% 0%
primitives/src/xcm 65% 0%
Summary 79% (3730 / 4738) 0% (0 / 0)

Minimum allowed line rate is 50%

Copy link
Contributor

@ermalkaleci ermalkaleci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Dinonard
Copy link
Member Author

@ipapandinas can you please repeat the approval?

Copy link
Contributor

@ipapandinas ipapandinas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Dinonard Dinonard merged commit d72239b into master Nov 21, 2024
9 checks passed
@Dinonard Dinonard deleted the feat/astar-governance branch November 21, 2024 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astar Related to Astar runtime This PR/Issue is related to the topic “runtime”. shibuya related to shibuya
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants