-
Notifications
You must be signed in to change notification settings - Fork 1k
Affluent vaults: update #2148
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Affluent vaults: update #2148
Conversation
|
The affluent adapter exports pools: Test Suites: 1 passed, 1 total |
@bhjiwonoh problem with that is we parse the symbol and use that to assign meta data, eg if its a stablecoin, single/multi exposure, etc... so changing it to pool names isn't great. you can however add the name to |
|
The affluent adapter exports pools: Test Suites: 1 passed, 1 total |
|
@slasher125 Separately, I have a question — could you explain how the "30d Avg APY" is calculated? |
|
@slasher125 I’ll add the logic to apply the most appropriate symbol and reopen the PR |
we calculate it server side, just a moving average of the last 30day apy values |
Changes
I realized that listing pool symbols as per the guideline doesn’t seem suitable when viewed on the dashboard, so I’ve changed it to use name instead. Since our vaults can change their constituent assets, using a unique name seems more appropriate. Please let me know if you see any issues with this change.