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

Trade History of markets is being sorted incorrectly in app. #19

Open
MellowYarker opened this issue Jul 12, 2021 · 1 comment
Open
Assignees
Labels
bug Something isn't working

Comments

@MellowYarker
Copy link
Owner

I haven't checked in a while, and I won't until later because I'm busy right now, but I think we're sorting by filler_oid, rather than by execution time.

@MellowYarker MellowYarker added the bug Something isn't working label Jul 12, 2021
@MellowYarker MellowYarker self-assigned this Jul 12, 2021
@MellowYarker
Copy link
Owner Author

This suffers from the same issue as account associated trades did. We need to store these trades in Redis, probably just a list of trades:market, where we add to the end when new trades come, and pop off the front once we know they're in postgres.

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
None yet
Development

No branches or pull requests

1 participant