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

Cast vote on proposal creation #799

Merged
merged 3 commits into from
Feb 9, 2024
Merged

Cast vote on proposal creation #799

merged 3 commits into from
Feb 9, 2024

Conversation

NoahSaso
Copy link
Member

@NoahSaso NoahSaso commented Feb 1, 2024

Resolves #798

This allows the proposer to cast a vote when creating a proposal, which also takes care of instant-passing proposals when the member with majority voting power proposes something (#759). It will fail if the proposer is not a member (when anyone may propose is enabled).

It works by simply chaining the vote execution function after the proposal creation. This makes it future proof by respecting all other relevant features (such as min_voting_period and allow_revoting, which would have been ignored by #776).

Copy link

codecov bot commented Feb 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7cc41b5) 96.17% compared to head (5b4cb1b) 96.21%.

Additional details and impacted files
@@               Coverage Diff               @@
##           development     #799      +/-   ##
===============================================
+ Coverage        96.17%   96.21%   +0.03%     
===============================================
  Files              207      207              
  Lines            53401    53909     +508     
===============================================
+ Hits             51359    51866     +507     
- Misses            2042     2043       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@JakeHartnell JakeHartnell left a comment

Choose a reason for hiding this comment

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

Wonderful! Really like this approach and that it saves a whole transaction!

Copy link
Collaborator

@bekauz bekauz left a comment

Choose a reason for hiding this comment

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

lgtm! awesome pr, only have a few small questions

@JakeHartnell
Copy link
Member

CI for integrations testing is failing, but there is a fix for that here: #715

@NoahSaso
Copy link
Member Author

NoahSaso commented Feb 6, 2024

CI for integrations testing is failing, but there is a fix for that here: #715

sir that is a bounties contract 😂

can you split up the CI fix into a separate pr pls <3

@JakeHartnell
Copy link
Member

Le fix for CI: #801

@NoahSaso
Copy link
Member Author

NoahSaso commented Feb 7, 2024

Le fix for CI: #801

thank you!

@NoahSaso
Copy link
Member Author

NoahSaso commented Feb 7, 2024

hey @JakeHartnell seems the CI is failing for a cache issue? how do we resolve it D:

@JakeHartnell
Copy link
Member

hey @JakeHartnell seems the CI is failing for a cache issue? how do we resolve it D:

Yolo merge? Ran test tube locally and everything passed so 99% sure it will work...

@NoahSaso NoahSaso merged commit 43eccde into development Feb 9, 2024
8 of 9 checks passed
@NoahSaso NoahSaso deleted the noah/auto-vote branch February 9, 2024 01:36
@NoahSaso NoahSaso mentioned this pull request Feb 21, 2024
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.

Allow auto casting a vote on proposal creation
3 participants