curve-contract icon indicating copy to clipboard operation
curve-contract copied to clipboard

feat: FRAX / USDC Pool

Open skellet0r opened this issue 2 years ago • 0 comments

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)
  • [x] 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

  • [x] I have included a pooldata.json with accurate token addresses
  • [x] I have included a README.md with accurate token addresses
  • [x] I have added a workflow targetting this pool, and confirmed that all tests are passing within the CI
  • [x] I have run brownie test tests/forked --network mainnet-fork locally and confirmed that all tests are passing
  • [x] 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

skellet0r avatar Jun 08 '22 10:06 skellet0r