-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Update Governance Spec: Tally Votes at End #1060
Comments
I'll update the spec. Note that this means we have to iterate over delegators vote. Previously we wanted to avoid that, and this was the main reason for the added complexity of the previous spec. |
@sunnya97 do we have the latest governance spec now? |
Update: The spec has not been updated yet because this proposal was conflicting with the urgent process (reminder: urgent process allowed a proposal to be accepted early if 2/3rd of the total voting power voted YES). For now, we are keeping tally at the end and removing urgent process from MVP. Urgency can be handled with Circuit Breakers (cf: #926). @kidinamoto01 the spec will be updated shortly |
@gamarin2 Good to know. |
Should we close this? @gamarin2 |
Yes! |
* Limit recursion depth for unknown field detection (cherry picked from commit f038dc731c55be1e1c526e67695acc358631afd6) * Limit unpack any (cherry picked from commit 1a2bff56fb7391f9ce87d4fbe9e0367ae991c0b2) * Update Changelog * Another limit recursion depth for unknown field detection * Update changelog Co-authored-by: Alexander Peters <[email protected]>
* Limit recursion depth for unknown field detection (cherry picked from commit f038dc731c55be1e1c526e67695acc358631afd6) * Limit unpack any (cherry picked from commit 1a2bff56fb7391f9ce87d4fbe9e0367ae991c0b2) * Update Changelog * Another limit recursion depth for unknown field detection * Update changelog Co-authored-by: Alexander Peters <[email protected]>
Vote tallying will be done at the end of the voting period for a proposal, rather than as votes are submitted. This will remove a lot of the odd UX stuff around voting (such as if you're changed your delegation, you're ineligible to vote for the next voting period).
The text was updated successfully, but these errors were encountered: