fix(dao): show LightChain's real staked-excluded quorum, not '3% of supply'#148
Merged
Conversation
…upply' The Governor overrides quorum() to use INativeVotes.getTotalVotingPower (the staked-excluded voting supply), so quorum is 3% of the votable base, not 3% of raw supply - worker and validator stake are excluded. The overview API now reads the actual quorum() + getTotalVotingPower() for LightChain, and the header shows the real threshold (~299M LCAI) with a '3% of the votable supply (staked LCAI excluded)' hint instead of the misleading '3% of supply'.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The governance header shows Quorum: 3% of supply, which is wrong for LightChain. The
LightChainGovernoroverridesquorum()to useINativeVotes.getTotalVotingPower(the staked-excluded voting supply), notgetPastTotalSupply. So quorum is 3% of the votable base - worker and validator stake are excluded. Verified on-chain: quorum = 298,972,950 LCAI = 3% of (getPastTotalSupply 9,966,500,000 - 735,000 net worker stake); the raw 10B supply also excludes ~33.5M validator bonded stake.Fix
app/api/dao-overview: for LightChain, also read the livequorum(clock-1)+getTotalVotingPower(clock-1)and returnquorumWei,votableSupplyWei,stakeExcludedFromQuorum.treasury-bar.tsx: show the real quorum amount (~299M LCAI) with a "3% of the votable supply (staked LCAI excluded)" hint instead of "3% of supply". Ethereum path unchanged (falls back to the % label).Data source is the same registries the SDK's
NativeGovernancereads (lightnode-sdk 0.21.0). Typecheck clean for both changed files.