Pass state instead

Author: Gonals

src/libs/actions/BankAccounts.js

@@ -335,14 +335,16 @@ function fetchUserWallet() {
  * Fetch the bank account currently being set up by the user for the free plan if it exists.
  *
  * @param {String} [stepToOpen]
- * @param {Boolean} [hasLocalOpenVBA]
+ * @param {String} [localBankAccountState]
  */
-function fetchFreePlanVerifiedBankAccount(stepToOpen, hasLocalOpenVBA = false) {
+function fetchFreePlanVerifiedBankAccount(stepToOpen, localBankAccountState) {
     // Remember which account BankAccountStep subStep the user had before so we can set it later
     const subStep = lodashGet(reimbursementAccountInSetup, 'subStep', '');
     const initialData = {loading: true, error: ''};
-    if (hasLocalOpenVBA) {
-        initialData.achData = {state: BankAccount.STATE.OPEN};
+
+    // Some UI needs to know the bank account state during the loading process, so we are keeping it in Onyx if passed
+    if (localBankAccountState) {
+        initialData.achData = {state: localBankAccountState};
     }
 
     // We are using set here since we will rely on data from the server (not local data) to populate the VBA flow

src/pages/workspace/WorkspacePageWithSections.js

@@ -47,8 +47,7 @@ const defaultProps = {
 class WorkspacePageWithSections extends React.Component {
     componentDidMount() {
         const achState = lodashGet(this.props.reimbursementAccount, 'achData.state', '');
-        const hasVBA = achState === BankAccount.STATE.OPEN;
-        fetchFreePlanVerifiedBankAccount('', hasVBA);
+        fetchFreePlanVerifiedBankAccount('', achState);
     }
 
     render() {