on master help sign in
Fix bug where entire application re-renders on setTxnConfirmations
commit8fa06596c4b2a730d15e3aa09e4ecbefad975550 authorpkova <pyry.kovanen@gmail.com> dateThu 25 Apr 2019, 23:45 (8 years ago) parent9e4c641
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() {