search

yearn / yearn-finance

🏦 yearn v2 web interface
https://yearn.finance
MIT License
167 stars 110 forks source link

fix: utilize new API for yvBOOST and UNI [WEB-407] #373

Closed ghost closed 3 years ago

linear[bot] commented 3 years ago
WEB-407 Update yvBOOST APY calculation in new API

@dudesahn do you have more context here? Currently the v2 website is using yvBOOST APY calculation from the old API

yearn-integrations commented 3 years ago

Important changes detected (payment related code)

vercel[bot] commented 3 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/y-finance/yearn-finance/FBib9w8rvXjQccp4Hu3kxBR6XVoC
✅ Preview: https://yearn-finance-git-dev-x48web-407-y-finance.vercel.app

dudesahn commented 3 years ago

yveCRV TVL isn't showing up now, even though it is working fine in the API.

Screen Shot 2021-06-23 at 10 40 02 PM

Screen Shot 2021-06-23 at 10 40 36 PM

dudesahn commented 3 years ago

My issue above seems to only be if you use the vercel preview, but pr-373.yearn.dev/vaults works fine Screen Shot 2021-06-23 at 10 45 03 PM

However, this TVL doesn't match what's in the API, so idk wtf is going on lol

yearn-integrations commented 3 years ago

Important changes detected (payment related code)