Merge pull request #28815 from MrMuzyk/migrateOnfidoWebsiteToFunctional

feat: migrate onfido web component from class to functioncal
Expensify · Oct 5, 2023 · 4078503 · 4078503
2 parents 56ced22 + da85209
commit 4078503
Showing 1 changed file with 23 additions and 22 deletions.
diff --git a/src/components/Onfido/index.website.js b/src/components/Onfido/index.website.js
@@ -1,34 +1,35 @@
-import React, {Component} from 'react';
+import React, {useEffect, useRef} from 'react';
 import lodashGet from 'lodash/get';
 import BaseOnfidoWeb from './BaseOnfidoWeb';
 import onfidoPropTypes from './onfidoPropTypes';
 
-class Onfido extends Component {
-    constructor(props) {
-        super(props);
-        this.baseOnfido = null;
-    }
+function Onfido({sdkToken, onSuccess, onError, onUserExit}) {
+    const baseOnfidoRef = useRef(null);
 
-    componentWillUnmount() {
-        const onfidoOut = lodashGet(this, 'baseOnfido.onfidoOut');
-        if (!onfidoOut) {
-            return;
-        }
+    useEffect(
+        () => () => {
+            const onfidoOut = lodashGet(baseOnfidoRef.current, 'onfidoOut');
+            if (!onfidoOut) {
+                return;
+            }
 
-        onfidoOut.tearDown();
-    }
+            onfidoOut.tearDown();
+        },
+        [],
+    );
 
-    render() {
-        return (
-            <BaseOnfidoWeb
-                ref={(e) => (this.baseOnfido = e)}
-                // eslint-disable-next-line react/jsx-props-no-spreading
-                {...this.props}
-            />
-        );
-    }
+    return (
+        <BaseOnfidoWeb
+            ref={baseOnfidoRef}
+            sdkToken={sdkToken}
+            onSuccess={onSuccess}
+            onError={onError}
+            onUserExit={onUserExit}
+        />
+    );
 }
 
 Onfido.propTypes = onfidoPropTypes;
+Onfido.displayName = 'Onfido';
 
 export default Onfido;