Skip to content

Commit

Permalink
Error boundary - sentry clean-up (#1915)
Browse files Browse the repository at this point in the history
* more sentry cleanup

* Optional chaining
  • Loading branch information
andrepimenta authored Oct 30, 2020
1 parent b0f0bcf commit 6d92e5f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion app/components/UI/Notification/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -461,6 +461,7 @@ class Notification extends PureComponent {
const { status } = this.props;
const { tx, transactionDetailsIsVisible, inBrowserView } = this.state;
const isPaymentChannelTransaction = tx && tx.paymentChannelTransaction;
const data = tx ? { ...tx.transaction, ...this.props.transaction } : { ...this.props.transaction };
return (
<ElevatedView
style={[
Expand All @@ -478,7 +479,7 @@ class Notification extends PureComponent {
>
<BaseNotification
status={status}
data={{ ...tx.transaction, ...this.props.transaction }}
data={data}
onPress={this.onNotificationPress}
onHide={this.onClose}
/>
Expand Down
2 changes: 1 addition & 1 deletion app/components/Views/Onboarding/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -637,7 +637,7 @@ class Onboarding extends PureComponent {
}

const mapStateToProps = state => ({
selectedAddress: state.engine.backgroundState.PreferencesController.selectedAddress,
selectedAddress: state?.engine?.backgroundState?.PreferencesController?.selectedAddress,
accounts: state.engine.backgroundState.AccountTrackerController.accounts,
passwordSet: state.user.passwordSet
});
Expand Down

0 comments on commit 6d92e5f

Please sign in to comment.