t11s
t11s
The problem is what event do we log the user?
🔍 Consider removing harvest delay logic and just have one var used only to guesstimate locked profit
Mentioned in the Yearn TOB audit (https://github.com/milkyklim/yearn-security/blob/07fca80ed196eb46821ba9e9bfc54f6d66ae328f/audits/20210719_ToB_yearn_vaultsv2/ToB_-_Yearn_Vault_v_2_Smart_Contracts_Audit_Report.pdf) and UniV2 Dapp.org (https://dapp.org.uk/reports/uniswapv2.html)
Need to write up a spec on what we want
- What to do if there's insolvency in a strategy (seizeStrategy() -> redeem as much as we can -> send underlying back to Vault) - What to do if a...
We need to do what Optics does and only grab the first 256 bits of return data