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

Enforce a fixed range for new Incentives #220

Open
guil-lambert opened this issue Dec 5, 2021 · 1 comment
Open

Enforce a fixed range for new Incentives #220

guil-lambert opened this issue Dec 5, 2021 · 1 comment

Comments

@guil-lambert
Copy link

This is related to this post on the Uniswap governance forum about incentivizing liquidity on L2's: https://gov.uniswap.org/t/consensus-check-should-uniswap-incentivize-liquidity-on-optimism-and-arbitrum/15288

I've created a fork of the v3 staker contract that adds a new minWidth parameter to the IncentiveKey. That way, new LM incentives may require that LP tokens have a minimum width (in ticks) in order to be staked. The goal is to limit whales from deploying 1-tick liquidity and collect an outsized portion of all LM rewards.

Here's the link to my fork: https://github.com/guil-lambert/v3-staker-fixed-range.

I'm no solidity dev and I don't want to do a pull request because it fails with I do yarn test (although the issues are not related to the smart contract per se). Just throwing this out there in case someone wants to create a proper fork/pull request.

@bigboydiamonds
Copy link

bigboydiamonds commented Mar 15, 2022

Great post and approach @guil-lambert!

We're currently implementing your approach with adding a minWidth parameter to Incentive Key struct in our fork: https://github.com/overlay-market/v3-staker/tree/jonah

I've updated the existing test suites to pass without issues following your code additions. Also included few more tests checking for the new require statements added in.

Do you suggest any additional implementations along with the minWidth parameter?

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

No branches or pull requests

2 participants