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

Add include submaccounts flag to listPositions #11650

Closed
JonRay15 opened this issue Sep 4, 2024 · 1 comment · Fixed by #11656
Closed

Add include submaccounts flag to listPositions #11650

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

Comments

@JonRay15
Copy link

JonRay15 commented Sep 4, 2024 •
edited
Loading

We requested this boolean flag.

It appears to have been added to the websocket.

https://docs.vega.xyz/testnet/api/rest/data-v2/trading-data-service-observe-positions

But it doesnt seem to be on the normal REST API and it ideally would be so that ppl (me) can debug.

https://docs.vega.xyz/testnet/api/rest/data-v2/trading-data-service-list-all-positions
@JonRay15 JonRay15 added the fe-request A list of reasonably high value API change requests from front end label Sep 4, 2024
@wwestgarth wwestgarth self-assigned this Sep 4, 2024
@wwestgarth wwestgarth mentioned this issue Sep 4, 2024
Merged
@JonRay15
Copy link
Author

JonRay15 commented Sep 9, 2024

Tested in FG and working.

@JonRay15 JonRay15 removed the fe-request A list of reasonably high value API change requests from front end label Sep 9, 2024
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