feat: update v2 frontend to use the new yveCRV -> 3Crv vault zap [WEB-467]

vercel[bot] commented 3 years ago

@patcito is attempting to deploy a commit to the Yearn Finance Team on Vercel.

A member of the Team first needs to authorize it.

yearn-integrations commented 3 years ago

Important changes detected (payment related code)

Found constant VYPER_ADDRESS in files app/components/BackscratcherClaim/index.js (-import { VYPER_ADDRESS, THREECRV_ADDRESS } from 'containers/Vaults/constants';), app/components/BackscratcherClaim/index.js (- const vyperContract = useContract(VYPER_ADDRESS);)
Found constant THREECRV_ADDRESS in files app/components/BackscratcherClaim/index.js (-import { VYPER_ADDRESS, THREECRV_ADDRESS } from 'containers/Vaults/constants';), app/components/BackscratcherClaim/index.js (+import { THREECRV_ADDRESS, ZAP_V2_3CRV } from 'containers/Vaults/constants';), app/containers/App/saga.js (+ addresses: [THREECRV_ADDRESS],)
Found keyword cacheSend in files app/containers/Vaults/saga.js (+ yield call(zapv2ThreeCrvContract.methods.zap.cacheSend, {)
Found address 0x579422a1c774470ca623329c69f27cc3beb935a1 in files app/containers/Vaults/constants.js. This address was ⚠️not found⚠️ in any vaults from https://vaults.finance/all

dudesahn commented 3 years ago

Checked and deposits still work as well, @x48-crypto or @xgambitox please review the code sers