-
Notifications
You must be signed in to change notification settings - Fork 504
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
Improve txList processing #1355
Conversation
TXNotes.selectNote(hash, Wallet.selectTxNotes(wallet)) || | ||
getAddressLabel(to, state).getOrElse('') | ||
|
||
const getPartnerLabel = hash => getBuySellTxHashMatch(state, hash) |
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.
Description/Label should possibly be moved to another service/selector
…page is still loading, fix pagi
@tony-blockchain need some help with the provider tests, the closest I got was in bch sagas spec, it's the state change test that is really tough. |
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.
let me
- check wether we can reactively select the labels
- uncomment and update tests
Description
Trying to improve performance of txlist. This moves the processing logic from the selector to the saga.
Change Type
Please enter one or more of the following:
Testing Steps
Use a heavy wallet with a lot of transactions and accounts
Code Checklist
yarn start
)README.md
and other documentation is updated as needed