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

feat: FRAX / USDC Pool #154

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

feat: FRAX / USDC Pool #154

wants to merge 7 commits into from

Conversation

skellet0r
Copy link
Member

New Pool Checklist

Pool Info

Tokens

FRAX

General Info

Behaviours

  1. On a successful transfer this token: True
  2. On a failed transfer this token: reverts
  3. Are there any special approval behaviors? None
  4. Is there a transfer fee, or the capability to add a transfer fee in the future? No (only minting/redemption fees)
  • I have reviewed the documentation and source code for this token, checking for unexpected behaviours that could affect the pool.

Incenvites

This pool WILL receive incentives other than CRV.

Fees

  1. Does this pool include tokens that are already handled by the fee burning process? Yes, USDC and FRAX are both already handled by the fee burner
  2. Does this pool include tokens that are not currently being burned, but which can be handled by an already deployed burner? N/A
  3. Does this pool require deployment of any new burners? N/A

Pre-review Checklist

  • I have included a pooldata.json with accurate token addresses
  • I have included a README.md with accurate token addresses
  • I have added a workflow targetting this pool, and confirmed that all tests are passing within the CI
  • I have run brownie test tests/forked --network mainnet-fork locally and confirmed that all tests are passing
  • I have verified the fee burning process for this pool and opened a related pull request on curve-dao-contracts if needed

Reviewer Pre-deployment Checklist

  • This PR contains a new workflow file, and the entire test suite is passing
  • I have verified the success / failure behaviours for each token in this pool
  • I have run brownie test tests/forked --network mainnet-fork locally and confirmed that all tests are passing
  • I have verified the fee burning process for this pool and any related pull requests
  • This pool makes no significant modifications to the core math OR has been reviewed personally by @michwill

Post-deployment Checklist

  • I have deployed a LiquidityGaugeV5 contract for this pool
  • I have deployed any required fee burner contracts
  • I have added pool and gauge deployment addresses to the pooldata.json and README.md files within this PR
  • I have opened a pull request on curve-docs to add the new deployment addresses to the documentation
  • I have created a DAO vote to add the new gauge and enable/configure fee burning

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.

None yet

1 participant