beefy-v2
beefy-v2 copied to clipboard
Metapool LP breakdowns
This adds support for metapool LP breakdowns. Any nested LPs will be replaced by it's held tokens in the display. I think it could be nice to continue showing the nested LP in our breakdown, but that was more UI work than I felt like doing.
looks great. what if we show underlying tokens (without am*)? can do this by enforcing "underlyingToken" property in all curvePools and then add it to API breakdown and match in app.
looks great. what if we show underlying tokens (without am*)? can do this by enforcing "underlyingToken" property in all curvePools and then add it to API breakdown and match in app.
Are you proposing "underlyingToken" as a new field? I don't see a reference to that anywhere in the code. If we're adding something new, it may make more sense to add "underlyingToken" in the address-book so that we can set something up that's not just Curve specific. I could see this being helpful for wTokens for example
new field "underlyingToken" to the lps/breakdown API. app will have to match underlyingToken || token for breakdown. those tokens are already in the address-book.
I've updated the display to show the asset tree. I've also updated the token name display, it still needs some polish, but let me know what you think.
This is really cool @BrknRobot.
@roman-monk wdyt?
yeah its very nice, i still think displaying underlying tokens as we do in filters and vault names would be more user friendly than "Aave Matic Market Dai"
I liked the technical detail, but you're probably right. I'll switch it over