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

Add Flexible Voting to Governor #4958

Closed
rsolari opened this issue Mar 15, 2024 · 4 comments · Fixed by #5045
Closed

Add Flexible Voting to Governor #4958

rsolari opened this issue Mar 15, 2024 · 4 comments · Fixed by #5045
Milestone

Comments

@rsolari
Copy link

rsolari commented Mar 15, 2024

🧐 Motivation

Scopelift's Flexible Voting extension (github) has been live for ~1.5 years. Several Governor implementations use it (Frax, Gitcoin, and PoolTogether) with more in the pipeline (Wormhole, Arbitrum, RadWorks).

The extension supports several new use cases, such as voting from a DeFi protocol and multichain voting. I have also talked to projects interested in building private voting or liquid delegation features on top of Flexible Voting.

I would be in favor of merging the Flexible Voting Extension into OZ Governor contracts to solidify the standard and make it easier for builders to build new use cases. I understand that there are other possible standards for fractional voting, but this one seems to be the one with by far the most traction.

📝 Details
The changes to support Flexible Voting are in this repository. If there is interest in this proposal, the ScopeLift team would probably be able to help advise on next steps for merging the code.

@rsolari rsolari changed the title Add FlexibleVoting to Governor Add Flexible Voting to Governor Mar 15, 2024
@apbendi
Copy link
Contributor

apbendi commented Mar 15, 2024

Hello, Ben from ScopeLift here 👋

Thanks, @rsolari, for suggesting this and notifying me that you had. Unsurprisingly, we are in favor of this be included in OZ. We see growing interest in Flexible Voting and the kinds of integrations it enables. I just wanted to add that we already have a PR upgrading the extension to work with OZ v5. We would be more than happy to migrate this to a PR against the main contracts repo and to work with the OZ team to integrate any feedback they would have to get it where it needs to be in order to be merged. Thanks in advance for the consideration!

@ernestognw ernestognw added this to the 5.x milestone Mar 18, 2024
@ernestognw
Copy link
Member

Hey, thanks @rsolari for putting everything together, and also thanks @apbendi for sharing your input here.

After reviewing ScopeLift's implementation, it seems to rely mostly on GovernorCountingFractional. In my opinion that'd be already a good first step. Would you say it unblock the use cases you're looking for?

I added the 5.x milestone given we already have work committed for the upcoming months and 5.1. Ideally, we'd take ScopeLift's implementation (tysm though!) but, of course it requires time for a review of both @Amxx and me, aside from some time to release.

@rsolari
Copy link
Author

rsolari commented Mar 19, 2024

Yes, GovernorCountingFractional would unlock the use cases I have in mind!

@ernestognw
Copy link
Member

For the record, this question came to the forum asking for the effect of delegations when depositing to a liquidity pool.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants