(Bug) #1052 Prevent to run multiple transaction

src/components/dashboard/SendQRSummary.js

@@ -55,6 +55,7 @@ const SendQRSummary = ({ screenProps }: AmountProps, params) => {
   const faceRecognition = () => {
     return screenProps.push('FRIntro', { from: 'SendQRSummary' })
   }
+
   const sendGD = () => {
     try {
       setLoading(true)
@@ -92,10 +93,15 @@ const SendQRSummary = ({ screenProps }: AmountProps, params) => {
             buttons: [{ text: 'Yay!' }],
             onDismiss: screenProps.goToRoot,
           })
+
+          setLoading(false)
+
           return hash
         },
         onError: e => {
           log.error('Send TX failed:', e.message, e)
+
+          setLoading(false)
           userStorage.markWithErrorEvent(txhash)
         },
       })
@@ -107,8 +113,6 @@ const SendQRSummary = ({ screenProps }: AmountProps, params) => {
         message: `There was a problem sending G$. Try again`,
         dismissText: 'OK',
       })
-    } finally {
-      setLoading(false)
     }
   }
 

src/components/dashboard/__tests__/SendQRSummary.js

@@ -2,7 +2,7 @@ import React from 'react'
 import renderer from 'react-test-renderer'
 import { getWebRouterComponentWithMocks } from './__util__'
 
-describe('OutOfGasError', () => {
+describe('SendQRSummary', () => {
   it('renders without errors', () => {
     const SendQRSummary = getWebRouterComponentWithMocks('../SendQRSummary')
     let component

src/components/dashboard/__tests__/__snapshots__/SendQRSummary.js.snap

@@ -1,6 +1,6 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`OutOfGasError matches snapshot 1`] = `
+exports[`SendQRSummary matches snapshot 1`] = `
 <div
   className="css-view-1dbjc4n"
   style={

src/components/profile/__tests__/OptionsRow.js

@@ -0,0 +1,19 @@
+import React from 'react'
+import renderer from 'react-test-renderer'
+import { withThemeProvider } from '../../../__tests__/__util__'
+import OptionsRow from '../OptionsRow'
+
+describe('ViewAvatar', () => {
+  const WrappedOptionsRow = withThemeProvider(OptionsRow)
+
+  it('renders without errors', () => {
+    const tree = renderer.create(<WrappedOptionsRow />)
+    expect(tree.toJSON()).toBeTruthy()
+  })
+
+  it('matches snapshot', () => {
+    const component = renderer.create(<WrappedOptionsRow />)
+    const tree = component.toJSON()
+    expect(tree).toMatchSnapshot()
+  })
+})

src/components/profile/__tests__/__snapshots__/OptionsRow.js.snap

@@ -0,0 +1,169 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`ViewAvatar matches snapshot 1`] = `
+<div
+  className="css-view-1dbjc4n"
+  style={
+    Object {
+      "WebkitBoxFlex": 1,
+      "WebkitFlexBasis": "0%",
+      "WebkitFlexGrow": 1,
+      "WebkitFlexShrink": 1,
+      "flexBasis": "0%",
+      "flexGrow": 1,
+      "flexShrink": 1,
+      "msFlexNegative": 1,
+      "msFlexPositive": 1,
+      "msFlexPreferredSize": "0%",
+    }
+  }
+>
+  <div
+    className="css-view-1dbjc4n"
+    style={
+      Object {
+        "WebkitAlignItems": "center",
+        "WebkitBoxAlign": "center",
+        "WebkitBoxDirection": "normal",
+        "WebkitBoxOrient": "horizontal",
+        "WebkitFlexDirection": "row",
+        "alignItems": "center",
+        "borderBottomColor": "rgba(238,238,238,1.00)",
+        "borderBottomStyle": "solid",
+        "borderBottomWidth": "1px",
+        "display": "flex",
+        "flexDirection": "row",
+        "msFlexAlign": "center",
+        "msFlexDirection": "row",
+        "paddingBottom": "8px",
+        "paddingLeft": "32px",
+        "paddingRight": "8px",
+        "paddingTop": "8px",
+      }
+    }
+  >
+    <div
+      className="css-text-901oao"
+      dir="auto"
+      style={
+        Object {
+          "WebkitBoxFlex": 2,
+          "WebkitFlexGrow": 2,
+          "color": "rgba(85,85,85,1.00)",
+          "direction": "ltr",
+          "flexGrow": 2,
+          "fontFamily": "Roboto",
+          "fontSize": "16px",
+          "fontWeight": "500",
+          "lineHeight": "22px",
+          "msFlexPositive": 2,
+          "textAlign": "left",
+          "textDecoration": "none",
+          "textTransform": "none",
+        }
+      }
+    >
+
+    </div>
+    <div
+      className="css-view-1dbjc4n"
+      style={
+        Object {
+          "WebkitAlignItems": "center",
+          "WebkitBoxAlign": "center",
+          "alignItems": "center",
+          "minWidth": "60px",
+          "msFlexAlign": "center",
+          "width": "15%",
+        }
+      }
+    >
+      <div
+        className="css-text-901oao"
+        dir="auto"
+        style={
+          Object {
+            "color": "rgba(85,85,85,1.00)",
+            "direction": "ltr",
+            "fontFamily": "Roboto",
+            "fontSize": "14px",
+            "fontWeight": "normal",
+            "lineHeight": "20px",
+            "textAlign": "center",
+            "textDecoration": "none",
+            "textTransform": "none",
+          }
+        }
+      >
+        Private
+      </div>
+    </div>
+    <div
+      className="css-view-1dbjc4n"
+      style={
+        Object {
+          "WebkitAlignItems": "center",
+          "WebkitBoxAlign": "center",
+          "alignItems": "center",
+          "minWidth": "60px",
+          "msFlexAlign": "center",
+          "width": "15%",
+        }
+      }
+    >
+      <div
+        className="css-text-901oao"
+        dir="auto"
+        style={
+          Object {
+            "color": "rgba(85,85,85,1.00)",
+            "direction": "ltr",
+            "fontFamily": "Roboto",
+            "fontSize": "14px",
+            "fontWeight": "normal",
+            "lineHeight": "20px",
+            "textAlign": "center",
+            "textDecoration": "none",
+            "textTransform": "none",
+          }
+        }
+      >
+        Masked
+      </div>
+    </div>
+    <div
+      className="css-view-1dbjc4n"
+      style={
+        Object {
+          "WebkitAlignItems": "center",
+          "WebkitBoxAlign": "center",
+          "alignItems": "center",
+          "minWidth": "60px",
+          "msFlexAlign": "center",
+          "width": "15%",
+        }
+      }
+    >
+      <div
+        className="css-text-901oao"
+        dir="auto"
+        style={
+          Object {
+            "color": "rgba(85,85,85,1.00)",
+            "direction": "ltr",
+            "fontFamily": "Roboto",
+            "fontSize": "14px",
+            "fontWeight": "normal",
+            "lineHeight": "20px",
+            "textAlign": "center",
+            "textDecoration": "none",
+            "textTransform": "none",
+          }
+        }
+      >
+        Public
+      </div>
+    </div>
+  </div>
+</div>
+`;

src/lib/gundb/UserStorageClass.js

@@ -1256,7 +1256,9 @@ export class UserStorage {
       feed
         .filter(
           feedItem =>
-            feedItem.data && ['deleted', 'cancelled'].includes(feedItem.status || feedItem.otplStatus) === false
+            feedItem &&
+            feedItem.data &&
+            ['deleted', 'cancelled'].includes(feedItem.status || feedItem.otplStatus) === false
         )
         .map(feedItem => {
           if (!(feedItem.data && feedItem.data.receiptData)) {