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.

Sign up for GitHub

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

Jump to bottom

[API] Liquidity fees paid doesnt like tye derived party flag #11655

Closed
JonRay15 opened this issue Sep 4, 2024 · 0 comments · Fixed by #11656
Closed

[API] Liquidity fees paid doesnt like tye derived party flag #11655

JonRay15 opened this issue Sep 4, 2024 · 0 comments · Fixed by #11656
Assignees
@wwestgarth

Comments

@JonRay15
Copy link

JonRay15 commented Sep 4, 2024

I seem to be getting a failure on the paid liquidity fees API when I try to set the include derived party flag to true.

Here I am querying for the party which has provided an AMM on the ETH market on FG and this falls over.

https://api.n00.testnet.vega.rocks/api/v2/liquidity/paidfees?partyIds=9c867b28b5a54e46639135669419d9bc521b36ed26632b75374c37137535b899&includeDerivedParties=true&epochFrom=235&epochTo=245

And here I have manually found the derived party and queried for it directly without setting that flag and it is fine.

https://api.n00.testnet.vega.rocks/api/v2/liquidity/paidfees?partyIds=b3fbd9b8a9c6cd19177e[…]dc64606ed0692b5c59a10e4990340&epochFrom=235&epochTo=245
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wwestgarth wwestgarth

Labels
None yet
Projects
Core Kanban
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add include-derived-parties flag to positions API vegaprotocol/vega
2 participants
@wwestgarth @JonRay15