Change error behavior of DeviceInteractionStep

src/components/DeviceInteraction/DeviceInteractionStep.js

@@ -10,7 +10,6 @@ import {
   SpinnerContainer,
   IconContainer,
   SuccessContainer,
-  ErrorDescContainer,
   ErrorContainer,
 } from './components'
 
@@ -20,10 +19,7 @@ export type Step = {
   desc?: React$Node,
   icon: React$Node,
   run?: Object => Promise<any> | { promise: Promise<any>, unsubscribe: void => any },
-  render?: (
-    { onSuccess: Object => any, onFail: Error => void, onRetry: void => void },
-    any,
-  ) => React$Node,
+  render?: ({ onSuccess: Object => any, onFail: Error => void }, any) => React$Node,
   minMs?: number,
 }
 
@@ -38,10 +34,8 @@ type Props = {
   isSuccess: boolean,
   isPassed: boolean,
   step: Step,
-  error: ?Error,
   onSuccess: (any, Step) => any,
   onFail: (Error, Step) => any,
-  onRetry: void => any,
   data: any,
 }
 
@@ -66,13 +60,13 @@ class DeviceInteractionStep extends PureComponent<
   }
 
   componentDidUpdate(prevProps: Props) {
-    const { isActive, error } = this.props
+    const { isActive, isError } = this.props
     const { status } = this.state
 
     const didActivated = isActive && !prevProps.isActive
     const didDeactivated = !isActive && prevProps.isActive
     const stillActivated = isActive && prevProps.isActive
-    const didResetError = !error && !!prevProps.error
+    const didResetError = !isError && !!prevProps.isError
 
     if (didActivated && status !== 'running') {
       this.run()
@@ -163,8 +157,6 @@ class DeviceInteractionStep extends PureComponent<
       isError,
       isPassed,
       step,
-      error,
-      onRetry,
       data,
     } = this.props
 
@@ -191,14 +183,8 @@ class DeviceInteractionStep extends PureComponent<
           )}
           {step.desc && step.desc}
           {CustomRender && (
-            <CustomRender
-              onSuccess={this.handleSuccess}
-              onFail={this.handleFail}
-              onRetry={onRetry}
-              data={data}
-            />
+            <CustomRender onSuccess={this.handleSuccess} onFail={this.handleFail} data={data} />
           )}
-          {isError && error && <ErrorDescContainer error={error} onRetry={onRetry} mt={2} />}
         </Box>
 
         <div style={{ width: 70, position: 'relative', overflow: 'hidden', pointerEvents: 'none' }}>

src/components/DeviceInteraction/components.js

@@ -92,6 +92,8 @@ export const ErrorContainer = ({ isVisible }: { isVisible: boolean }) => (
 )
 
 const ErrorRetryContainer = styled(Box).attrs({
+  bg: rgba(colors.alertRed, 0.1),
+  borderRadius: 1,
   grow: 1,
   color: 'alertRed',
   cursor: 'pointer',
@@ -118,6 +120,7 @@ export const ErrorDescContainer = ({
 }) => (
   <Box
     pl={0}
+    fontSize={3}
     color="alertRed"
     align="flex-start"
     justify="center"
@@ -126,7 +129,7 @@ export const ErrorDescContainer = ({
     }}
     {...p}
   >
-    <Box horizontal bg={rgba(colors.alertRed, 0.1)} borderRadius={1}>
+    <Box horizontal>
       <Box p={1} pl={2}>
         {error.message || 'Failed'}
       </Box>

src/components/DeviceInteraction/index.js

@@ -1,40 +1,43 @@
 // @flow
 
 import React, { PureComponent } from 'react'
-import styled from 'styled-components'
 
 import { delay } from 'helpers/promise'
 
 import Box from 'components/base/Box'
+import Space from 'components/base/Space'
+
 import DeviceInteractionStep from './DeviceInteractionStep'
 
 import type { Step } from './DeviceInteractionStep'
 
+import { ErrorDescContainer } from './components'
+
+type Props = {
+  steps: Step[],
+  onSuccess?: any => void,
+  onFail?: any => void,
+  waitBeforeSuccess?: number,
+
+  // when true and there is an error, display the error + retry button
+  shouldRenderRetry?: boolean,
+}
+
+type State = {
+  stepIndex: number,
+  isSuccess: boolean,
+  error: ?Error,
+  data: Object,
+}
+
 const INITIAL_STATE = {
   stepIndex: 0,
   isSuccess: false,
-  showSuccess: false,
   error: null,
   data: {},
 }
 
-class DeviceInteraction extends PureComponent<
-  {
-    steps: Step[],
-    onSuccess?: any => void,
-    onFail?: any => void,
-    renderSuccess?: any => any,
-    waitBeforeSuccess?: number,
-  },
-  {
-    stepIndex: number,
-    isSuccess: boolean,
-    // used to be able to display the last check for a small amount of time
-    showSuccess: boolean,
-    error: ?Error,
-    data: Object,
-  },
-> {
+class DeviceInteraction extends PureComponent<Props, State> {
   state = INITIAL_STATE
 
   componentWillUnmount() {
@@ -58,12 +61,11 @@ class DeviceInteraction extends PureComponent<
       if (!waitBeforeSuccess) {
         onSuccess && onSuccess(data)
       }
-      this.setState({ isSuccess: true, data, showSuccess: !waitBeforeSuccess })
+      this.setState({ isSuccess: true, data })
       if (waitBeforeSuccess) {
         await delay(waitBeforeSuccess)
         if (this._unmounted) return
         onSuccess && onSuccess(data)
-        this.setState({ showSuccess: true })
       }
     } else {
       this.setState({ stepIndex: stepIndex + 1, data })
@@ -82,39 +84,40 @@ class DeviceInteraction extends PureComponent<
   }
 
   render() {
-    const { steps, renderSuccess, waitBeforeSuccess: _waitBeforeSuccess, ...props } = this.props
-    const { stepIndex, error, isSuccess, data, showSuccess } = this.state
+    const { steps, shouldRenderRetry, ...props } = this.props
+    const { stepIndex, error, isSuccess, data } = this.state
 
     return (
-      <DeviceInteractionContainer {...props}>
-        {isSuccess && showSuccess && renderSuccess
-          ? renderSuccess(data)
-          : steps.map((step, i) => {
-              const isError = !!error && i === stepIndex
-              return (
-                <DeviceInteractionStep
-                  key={step.id}
-                  step={step}
-                  error={isError ? error : null}
-                  isError={isError}
-                  isFirst={i === 0}
-                  isLast={i === steps.length - 1}
-                  isPrecedentActive={i === stepIndex - 1}
-                  isActive={i === stepIndex}
-                  isPassed={i < stepIndex}
-                  isSuccess={i < stepIndex || (i === stepIndex && isSuccess)}
-                  onSuccess={this.handleSuccess}
-                  onFail={this.handleFail}
-                  onRetry={this.reset}
-                  data={data}
-                />
-              )
-            })}
-      </DeviceInteractionContainer>
+      <Box {...props}>
+        {steps.map((step, i) => {
+          const isError = !!error && i === stepIndex
+          return (
+            <DeviceInteractionStep
+              key={step.id}
+              step={step}
+              isError={isError}
+              isFirst={i === 0}
+              isLast={i === steps.length - 1}
+              isPrecedentActive={i === stepIndex - 1}
+              isActive={i === stepIndex}
+              isPassed={i < stepIndex}
+              isSuccess={i < stepIndex || (i === stepIndex && isSuccess)}
+              onSuccess={this.handleSuccess}
+              onFail={this.handleFail}
+              data={data}
+            />
+          )
+        })}
+        {error &&
+          shouldRenderRetry && (
+            <Box align="flex-end">
+              <Space of={0} />
+              <ErrorDescContainer error={error} onRetry={this.reset} mt={2} />
+            </Box>
+          )}
+      </Box>
     )
   }
 }
 
-const DeviceInteractionContainer = styled(Box).attrs({})``
-
 export default DeviceInteraction

src/components/GenuineCheck.js

@@ -101,7 +101,7 @@ class GenuineCheck extends PureComponent<Props> {
   }
 
   render() {
-    const { onSuccess } = this.props
+    const { onSuccess, ...props } = this.props
     const steps = [
       {
         id: 'device',
@@ -146,6 +146,7 @@ class GenuineCheck extends PureComponent<Props> {
         steps={steps}
         onSuccess={onSuccess}
         onFail={this.handleFail}
+        {...props}
       />
     )
   }

src/components/ManagerPage/ManagerGenuineCheck.js

@@ -21,7 +21,8 @@ class ManagerGenuineCheck extends PureComponent<Props> {
   render() {
     const { t, onSuccess } = this.props
     return (
-      <Box align="center" justify="center" sticky style={{ marginRight: 80 }}>
+      <Box align="center">
+        <Space of={60} />
         <Box align="center" style={{ maxWidth: 460 }}>
           <img
             src={i('logos/connectDevice.png')}
@@ -36,15 +37,7 @@ class ManagerGenuineCheck extends PureComponent<Props> {
           </Text>
         </Box>
         <Space of={40} />
-        <GenuineCheck
-          onSuccess={onSuccess}
-          onFail={() => {
-            console.log(`fail`)
-          }}
-          onUnavailable={() => {
-            console.log(`unavailable`)
-          }}
-        />
+        <GenuineCheck shouldRenderRetry onSuccess={onSuccess} />
       </Box>
     )
   }

src/components/ManagerPage/index.js

@@ -22,9 +22,11 @@ type State = {
 class ManagerPage extends PureComponent<Props, State> {
   state = {
     isGenuine: null,
+    device: null,
+    deviceInfo: null,
   }
 
-  handleSuccessGenuine = ({ device, deviceInfo }) => {
+  handleSuccessGenuine = ({ device, deviceInfo }: { device: Device, deviceInfo: DeviceInfo }) => {
     this.setState({ isGenuine: true, device, deviceInfo })
   }