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

order param not working in /liquidity_pools/{id}/operations #3870

Closed
bartekn opened this issue Aug 30, 2021 · 0 comments · Fixed by #3872
Closed

order param not working in /liquidity_pools/{id}/operations #3870

bartekn opened this issue Aug 30, 2021 · 0 comments · Fixed by #3872
Assignees
@2opremio
Labels
amm support cap 38 (automated market makers) in horizon bug

Comments

@bartekn
Copy link
Contributor

bartekn commented Aug 30, 2021

What version are you using?

amm branch

What did you do?

http://localhost:8000/liquidity_pools/64e163b66108152665ee325cc333211446277c86bfe021b9da6bb1769b0daea1/operations?order=desc

(note I changed liquidity_pool -> liquidity_pools in my integration tests branch)

What did you expect to see?

Operations sorted by ledger time descending.

What did you see instead?

Operations sorted by ledger time ascending.
@bartekn bartekn added bug amm support cap 38 (automated market makers) in horizon labels Aug 30, 2021
@bartekn bartekn linked a pull request Aug 30, 2021 that will close this issue
services/horizon: Fix operation ordering when querying by claimable balance and liq pools #3872
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@2opremio 2opremio

Labels
amm support cap 38 (automated market makers) in horizon bug
Projects
None yet
Milestone
No milestone
2 participants
@bartekn @2opremio