Fix propTypes for RequestorStep

Expensify · Oct 18, 2022 · 75ab5b9 · 75ab5b9
1 parent e03a8d7
commit 75ab5b9
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 5 deletions.
diff --git a/src/pages/ReimbursementAccount/RequestorOnfidoStep.js b/src/pages/ReimbursementAccount/RequestorOnfidoStep.js
@@ -18,16 +18,21 @@ const propTypes = {
     /** Bank account currently in setup */
     /* eslint-disable-next-line react/no-unused-prop-types */
     reimbursementAccount: reimbursementAccountPropTypes.isRequired,
+
+    /** The draft values of the bank account being setup */
     /* eslint-disable-next-line react/no-unused-prop-types */
     reimbursementAccountDraft: reimbursementAccountDraftPropTypes.isRequired,
-    onfidoToken: PropTypes.string,
+
+    /** The token required to initialize the Onfido SDK */
+    onfidoToken: PropTypes.string.isRequired,
+
+    /** A callback to call once the user completes the Onfido flow */
     onComplete: PropTypes.func.isRequired,
+
     ...withLocalizePropTypes,
 };
 
-const defaultProps = {
-    onfidoToken: '',
-};
+const defaultProps = {};
 
 class RequestorOnfidoStep extends React.Component {
     constructor(props) {

diff --git a/src/pages/ReimbursementAccount/RequestorStep.js b/src/pages/ReimbursementAccount/RequestorStep.js
@@ -26,11 +26,16 @@ import * as Link from '../../libs/actions/Link';
 import RequestorOnfidoStep from './RequestorOnfidoStep';
 
 const propTypes = {
-    /** Bank account currently in setup */
+    /** The bank account currently in setup */
     reimbursementAccount: reimbursementAccountPropTypes.isRequired,
+
+    /** The draft values of the bank account being setup */
     /* eslint-disable-next-line react/no-unused-prop-types */
     reimbursementAccountDraft: reimbursementAccountDraftPropTypes.isRequired,
+
+    /** The token required to initialize the Onfido SDK */
     onfidoToken: PropTypes.string,
+
     ...withLocalizePropTypes,
 };