don't include new apy in gross apy

yearn / yearn-sdk

🦧 SDK for the yearn.finance platform. WIP

https://npm.im/@yfi/sdk

MIT License

53 stars 56 forks source link

Closed patcito closed 2 years ago

patcito commented 2 years ago

After depositing in Rocket Pool I got this APY. Thinking maybe we should remove new vault apy from gross apy. ryield

patcito commented 2 years ago

Some alternatives to this fix:

karelianpie commented 2 years ago

@patcito can you please rebase this with master so that we try and get the new GitHub action to run instead of trying to deploy to GitHub Pages?

patcito commented 2 years ago

@patcito can you please rebase this with master so that we try and get the new GitHub action to run instead of trying to deploy to GitHub Pages?

done

jstashh commented 2 years ago

Thanks for this @patcito, as an aside can you sign commits in future pls?