symbol / mobile-wallet

Symbol Mobile Wallet
Apache License 2.0
8 stars 14 forks source link

Application crash when adding account to multisig #319

Open cryptoBeliever opened 2 years ago

cryptoBeliever commented 2 years ago

Steps to reproduce

M1 - account that we want convert to multi-sig (desktop wallet) C1 - cosigner1 (mobile wallet) C2 - cosigner2 (desktop wallet)

  1. On mobile (account C1) go to transaction history view
  2. On desktop wallet start conversion to multisig account M1 using cosigners C1 and C2.
  3. On mobile wallet (C1) accept transaction and stay on transaction history view.
  4. On desktop wallet (C2) accept transaction

Result: When the transaction is confirmed mobile wallet crashes.

Error (from debug mode):

Stack:

ExceptionsManager.js:44 TypeError: Cannot read properties of undefined (reading 'transactionType')

This error is located at:
    in TransactionItem (created by ConnectFunction)
    in ConnectFunction (at History.js:100)
    in RCTView (at ListItem.js:33)
    in RCTView (at createAnimatedComponent.js:151)
    in AnimatedComponent (at TouchableOpacity.js:308)
    in TouchableOpacity (at ListItem.js:32)
    in GradientBackground (at History.js:99)
    in RCTView (at VirtualizedList.js:1925)
    in CellRenderer (at VirtualizedList.js:767)
    in RCTView (at ScrollView.js:1038)
    in RCTScrollView (at ScrollView.js:1166)
    in AndroidSwipeRefreshLayout (at RefreshControl.js:186)
    in RefreshControl (at History.js:174)
    in ScrollView (at VirtualizedList.js:1183)
    in VirtualizedList (at FlatList.js:676)
    in FlatList (at History.js:165)
    in RCTView (at ListContainer.js:50)
    in RCTView (at Section.js:62)
    in Text (at ListContainer.js:49)
    in GradientBackground (at History.js:164)
    in RCTView (at Container.js:25)
    in ComponentName (at GradientBackground.js:131)
    in RCTView (at react-native-linear-gradient/index.android.js:84)
    in LinearGradient (at GradientBackground.js:122)
    in RCTView (at createAnimatedComponent.js:151)
    in AnimatedComponent (at FadeView.js:31)
    in FadeView (at GradientBackground.js:121)
    in GradientBackground (at History.js:130)
    in History (created by ConnectFunction)
    in ConnectFunction (at Dashboard.js:68)
    in RCTView (at Dashboard.js:67)
    in RCTView (at react-native-linear-gradient/index.android.js:84)
    in LinearGradient (at GradientBackground.js:122)
    in RCTView (at createAnimatedComponent.js:151)
    in AnimatedComponent (at FadeView.js:31)
    in FadeView (at GradientBackground.js:121)
    in GradientBackground (at Dashboard.js:66)
    in Dashboard (created by ConnectFunction)
    in ConnectFunction (at gestureHandlerRootHOC.js:13)
    in GestureHandlerRootView (at GestureHandlerRootView.android.js:31)
    in GestureHandlerRootView (at gestureHandlerRootHOC.js:12)
    in gestureHandlerRootHOC(Connect(Dashboard)) (at Router.js:131)
    in Provider (at Router.js:130)
    in EnhancedComponent (at Router.js:134)
    in inject (at ComponentWrapper.js:37)
    in WrappedComponent (at renderApplication.js:40)
    in RCTView (at AppContainer.js:101)
    in RCTView (at AppContainer.js:119)
    in AppContainer (at renderApplication.js:39)

Current behavior The application crashes when the transaction is confirmed.

Expected behavior The application is not crashing, the view is refreshed and the transaction is presented as confirmed.

Affected Version Tested on the dev branch.

Screenshots image