-
Notifications
You must be signed in to change notification settings - Fork 414
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
persists transaction filter: RE #1154 #2048
Conversation
c926d3b
to
be237d6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just the change to add an "all"
transaction type. To have this properly persist you will need to add a new redux filter like this: https://github.com/lbryio/lbry-desktop/blob/master/src/renderer/store.js#L108
We only need to persist the filter setting from that reducer.
Other than that looks good! This will be great to have.
be237d6
to
3b58914
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two minor things. Could you add a changelog update too?
3b58914
to
c74def2
Compare
DEPENDS ON: lbryio/lbry-redux#84
ISSUE: #1154
Filter choices in transaction list persist.