Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

[beta] Token filter balances (throttle) #7742

Merged
merged 5 commits into from
Jan 31, 2018
Merged

[beta] Token filter balances (throttle) #7742

merged 5 commits into from
Jan 31, 2018

Conversation

jacogr
Copy link
Contributor

@jacogr jacogr commented Jan 30, 2018

  • Split token retrievals into 128-item sized chunks.
  • Align parity-js/shared for shell
  • Tested with cargo build --release --=no-default-features --features ui

parity 2018-01-31 10-22-00

@jacogr jacogr added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. A8-backport 🕸 Pull request is already reviewed well in another branch. M7-ui labels Jan 30, 2018
jacogr added a commit to parity-js/shared that referenced this pull request Jan 31, 2018
jacogr added a commit to parity-js/shared that referenced this pull request Jan 31, 2018
@jacogr jacogr added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Jan 31, 2018
Copy link
Contributor

@5chdn 5chdn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Compiled and tested locally - displays token balances correctly.

@5chdn
Copy link
Contributor

5chdn commented Jan 31, 2018

Can we also have this fix in stable?

@5chdn 5chdn added the B0-patch label Jan 31, 2018
@5chdn 5chdn added this to the Patch milestone Jan 31, 2018
@5chdn 5chdn merged commit 73be0fb into beta Jan 31, 2018
@5chdn 5chdn deleted the jg-beta-balances branch January 31, 2018 13:59
jacogr added a commit to parity-js/dapp-wallet that referenced this pull request Feb 5, 2018
jacogr added a commit to parity-js/dapp-wallet that referenced this pull request Feb 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-pleasereview 🤓 Pull request needs code review. A8-backport 🕸 Pull request is already reviewed well in another branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants