on master help sign in
Merge pull request #126 from pkova/master

Fix bug where entire application re-renders on setTxnConfirmations

commit7ef9095cdaf7dd6d15929ac6c2aa01ce9aebdfd4 authorFang <Fang-@users.noreply.github.com> dateThu 25 Apr 2019, 23:19 (8 years ago) parent9e4c6418fa0659
1 file changed+6−3
src/bridge/Bridge.js+6−3
src/bridge/Bridge.js+6−3
@@ -244,9 +244,12 @@ class Bridge extends React.Component {
244244 }
245245
246246 setTxnConfirmations(txnHash, txnConfirmations) {
247− this.setState((prevState, _) =>
248− ({txnConfirmations: { ...prevState.txnConfirmations,
249− [txnHash]: txnConfirmations}}))
247+ this.setState((prevState, _) => {
248+ return prevState.routeCrumbs.peek() === ROUTE_NAMES.SENT_TRANSACTION
249+ ? {txnConfirmations: { ...prevState.txnConfirmations,
250+ [txnHash]: txnConfirmations}}
251+ : null
252+ })
250253 }
251254
252255 render() {