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

A convenient way to query all open lp positions from pcli #4825

Open
masonyonkers opened this issue Aug 21, 2024 · 2 comments
Open

A convenient way to query all open lp positions from pcli #4825

masonyonkers opened this issue Aug 21, 2024 · 2 comments
Assignees
Labels
A-client Area: Design and implementation for client functionality _P-medium Medium priority
Milestone

Comments

@masonyonkers
Copy link
Contributor

Is your feature request related to a problem? Please describe.
There is currently no easy way to view the status of all of a wallet's open liquidity positions from pcli.

Describe the solution you'd like
A subcommand that acts like pcli query dex position plpid for all open positions in an account.

Describe alternatives you've considered
I am currently using a bash script that queries balance and runs pcli query dex position plpid for each open position. This works for now, but is slow and not feasable if there are a large number of open positions.

Additional context
N/A

@github-actions github-actions bot added the needs-refinement unclear, incomplete, or stub issue that needs work label Aug 21, 2024
@aubrika aubrika added this to the Sprint 11 milestone Aug 23, 2024
@aubrika aubrika added A-client Area: Design and implementation for client functionality and removed needs-refinement unclear, incomplete, or stub issue that needs work labels Aug 23, 2024
@aubrika
Copy link
Contributor

aubrika commented Aug 23, 2024

Thanks for opening this, I think it's a good idea. We'll add it to our queue. :)

@zbuc
Copy link
Member

zbuc commented Aug 23, 2024

I agree that pcli should be changed to support displaying all positions at once.

@masonyonkers I do want to point out that depending on what you're trying to do you might be better off using the penumbra libraries from a Rust program instead of automating pcli with bash. You can see an example of it here: https://github.com/penumbra-zone/osiris

@aubrika aubrika added the _P-medium Medium priority label Aug 26, 2024
@aubrika aubrika modified the milestones: Sprint 11, Sprint 12 Sep 9, 2024
@aubrika aubrika modified the milestones: Sprint 12, Sprint 13 Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-client Area: Design and implementation for client functionality _P-medium Medium priority
Projects
Status: Backlog
Development

No branches or pull requests

4 participants