penumbra-zone / dex-explorer

Web app for visualization the state of the Penumbra DEX
Apache License 2.0
10 stars 1 forks source link

Positions should be shown ordered by effective price #32

Open hdevalence opened 3 months ago

hdevalence commented 3 months ago
Screenshot 2024-08-15 at 3 34 48 PM

The highlighted LP has a 5% fee tier (it has been extremely profitable!), but this isn't reflected in the ordering. The positions should be sorted by effective price instead, which would render it correctly. Also, the title on the sidebar should be changed to "Direct Liquidity Positions" since they're not actually orders, they're individual AMMs.