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

Memoize DataView deps #995

Merged
merged 6 commits into from
Sep 17, 2019
Merged

Memoize DataView deps #995

merged 6 commits into from
Sep 17, 2019

Conversation

AquiGorka
Copy link
Contributor

@AquiGorka AquiGorka force-pushed the feature/memoize-dataview-deps branch from bfbbcc9 to a5a45b0 Compare September 16, 2019 07:55
apps/finance/app/src/components/Transfers.js Outdated Show resolved Hide resolved
@coveralls
Copy link

Coverage Status

Coverage increased (+0.4%) to 98.009% when pulling aa093c1 on feature/memoize-dataview-deps into 65df4d0 on master.

Co-Authored-By: Pierre Bertet <hello@pierre.world>
@AquiGorka AquiGorka merged commit a7a256e into master Sep 17, 2019
@sohkai sohkai deleted the feature/memoize-dataview-deps branch November 8, 2019 13:46
MickdeGraaf pushed a commit to MickdeGraaf/aragon-apps that referenced this pull request Jan 28, 2020
* Transfers: memoize sorted transfers

* Transfers: update code format

* Transfers: memoize sorted transfers

* Transfers: update code format

* Holders: memoize holders

* Transfers: remove react namespace from method

Co-Authored-By: Pierre Bertet <hello@pierre.world>
ramilexe pushed a commit to ConsiderItDone/aragon-apps that referenced this pull request Dec 9, 2021
* Transfers: memoize sorted transfers

* Transfers: update code format

* Transfers: memoize sorted transfers

* Transfers: update code format

* Holders: memoize holders

* Transfers: remove react namespace from method

Co-Authored-By: Pierre Bertet <hello@pierre.world>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants