Add "Trading APY" to our overall APY per vault. ($500 UI + $2000 API Bounty)
sirbeefalot opened this issue · 3 comments
This is a feature that I've been meaning to add for some time, and I just saw that Alpha Homora has it implemented.
They display the trading APY of holding an LP and add it to the total as well. This is relevant because a lot of the time, you're comparing the profitability of farming with an LP as opposed to holding an asset like BNB. In this sense, the trading profits are part of the equation, and it would be good to be explicit about it.
Blocked by API task: beefyfinance/beefy-api#111
Since pancake info data is outdated, we should have a warning, like Alpha has: "Trading fees APR shown has only synced to block number 5640759/6706829. The data shown may be delayed and not reflect the actual APR." Or we simply wait until pancake data is fixed to show their trading data, since it can be misleading.
Hi, Can you assign me this issue to me?
Of course 😄
Edit: Let me know if you need any help, or feel free to PM me over discord: @sirbeefalot