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

Allow filtering of DEX Trades by time range and address #1753

Closed
LefterisJP opened this issue Nov 20, 2020 · 1 comment · Fixed by #1886
Closed

Allow filtering of DEX Trades by time range and address #1753

LefterisJP opened this issue Nov 20, 2020 · 1 comment · Fixed by #1886
Assignees
@kelsos
Labels
frontend
Milestone
v1.10.0

Comments

@LefterisJP
Copy link
Member

Abstract

I was trying to find a particular trade and had to scroll through 400 pages of trades (oh boy that test account is crazy).

2020-11-20_02-43

It would really help the user if they can filter by time range and/or address

Task

Do it
@LefterisJP LefterisJP modified the milestones: v1.10.0, v1.9.1 Nov 20, 2020
@kelsos kelsos self-assigned this Nov 25, 2020
@LefterisJP LefterisJP modified the milestones: v1.9.1, v1.10.0 Nov 29, 2020
@LefterisJP
Copy link
Member Author

@kelsos this looks great. I just tested it. Will merge the premium components PR. To close the issue can you also add a changelog entry?

@kelsos kelsos mentioned this issue Dec 10, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kelsos kelsos

Labels
frontend
Projects
None yet
Milestone
v1.10.0
Development

Successfully merging a pull request may close this issue.

Updates changelog rotki/rotki
2 participants
@LefterisJP @kelsos