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

Governance: voting_cool_off_time #3830

Merged

Conversation

SebastianBor
Copy link
Contributor

@SebastianBor SebastianBor commented Nov 18, 2022

Summary

Introduce voting cool of time to add an extra time when Community and/or Council can disapprove or veto proposals.

Implementation Details

  • All existing Governances with max_voting_time > 10h default to 1h cool of time. The setting can be further adjusted or removed via config change proposal.
  • Cool of time doesn't apply when a vote ends early with Early or Strict tipping rules.

@SebastianBor SebastianBor marked this pull request as ready for review November 18, 2022 23:45
Copy link
Contributor

@joncinque joncinque left a comment

Choose a reason for hiding this comment

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

R+ after addressing Max's points

governance/program/src/state/governance.rs Outdated Show resolved Hide resolved
governance/program/src/state/proposal.rs Outdated Show resolved Hide resolved
@SebastianBor SebastianBor merged commit 21cfaff into solana-labs:master Nov 25, 2022
HaoranYi pushed a commit to HaoranYi/solana-program-library that referenced this pull request Jul 19, 2023
* wip: add voting_cool_off_time

* feat: Default voting_cool_off_time to 1h for 10h or longer votes

* wip: Assert valid voting cool of time

* chore: cleanup GovernanceConfig tests

* chore: Rename vote_end_time() to expected_vote_end_time()

* chore: test_cast_council_veto_vote_within_cool_off_time

* chore: test_cast_approve_vote_with_cannot_vote_in_cool_off_time_error

* chore: test_change_yes_vote_to_no_within_cool_off_time

* chore: Reorder GovernanceConfig fields

* chore: test_finalize_vote_with_cannot_finalize_during_cool_off_time_error

* fix: Remove default voting_cool_off_time

* fix: Use max = base + cool_off voting time setup

* chore: Cleanup

* chore: Fix test_change_yes_vote_to_no_within_cool_off_time

* chore: Adjust base, cool off and max voting time names

* chore: Create has_voting_base_time_ended
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants