Skip to content
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

Implement top price mover on the explore page #371

Open
erwanor opened this issue Feb 13, 2025 · 1 comment
Open

Implement top price mover on the explore page #371

erwanor opened this issue Feb 13, 2025 · 1 comment
Labels
bug Something isn't working

Comments

@erwanor
Copy link
Member

erwanor commented Feb 13, 2025

We removed the top price mover info card in #370 because it was showing nonsensical values.

We should restore it, adjusting the query so that it filters results based on a configurable threshold of liquidity (e.g, 10k). I believe that most of the work is done (?), and we need to plumb it correctly.

cc @cronokirby for more context on what's left to do here, iirc a config flag PR got merged into pindexer, but the data is still bad, maybe it's not used yet?

@erwanor erwanor added the bug Something isn't working label Feb 13, 2025
@github-project-automation github-project-automation bot moved this to 🗄️ Backlog in Penumbra web Feb 13, 2025
@erwanor erwanor moved this from 🗄️ Backlog to 📝 Todo in Penumbra web Feb 13, 2025
@cronokirby
Copy link
Contributor

penumbra-zone/penumbra#4936 is the one adding that config in the pindexer cli, still unmerged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 📝 Todo
Development

No branches or pull requests

2 participants