From 2d86fa4b5ea2b107f56e8c954d4f07064e569506 Mon Sep 17 00:00:00 2001 From: Gorka Ludlow Date: Tue, 17 Sep 2019 08:07:52 +0200 Subject: [PATCH] Memoize DataView deps (#995) * 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 --- app/src/screens/Holders.js | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/app/src/screens/Holders.js b/app/src/screens/Holders.js index 82070bd90d..5599e179e4 100644 --- a/app/src/screens/Holders.js +++ b/app/src/screens/Holders.js @@ -1,4 +1,4 @@ -import React, { useCallback } from 'react' +import React, { useMemo, useCallback } from 'react' import PropTypes from 'prop-types' import BN from 'bn.js' import { useConnectedAccount } from '@aragon/api-react' @@ -34,13 +34,18 @@ function Holders({ tokenTransfersEnabled, }) { const connectedAccount = useConnectedAccount() + const mappedEntries = useMemo( + () => holders.map(({ address, balance }) => [address, balance]), + [holders] + ) + return ( [address, balance])} + entries={mappedEntries} renderEntry={([address, balance]) => { const isCurrentUser = addressesEqual(address, connectedAccount)