Prevent Plaid triggered again on selector bank accounts page after ba…

…ck from offline

src/components/AddPlaidBankAccount.js

@@ -78,21 +78,19 @@ class AddPlaidBankAccount extends React.Component {
 
     componentDidMount() {
         // If we're coming from Plaid OAuth flow then we need to reuse the existing plaidLinkToken
-        if ((this.props.receivedRedirectURI && this.props.plaidLinkOAuthToken)
-            || !_.isEmpty(lodashGet(this.props.plaidData, 'bankAccounts'))
-            || !_.isEmpty(lodashGet(this.props.plaidData, 'errors'))) {
+        if ((this.props.receivedRedirectURI && this.props.plaidLinkOAuthToken) || this.hasPlaidBankAccountsData()) {
             return;
         }
 
         BankAccounts.openPlaidBankLogin(this.props.allowDebit, this.props.bankAccountID);
     }
 
     componentDidUpdate(prevProps) {
-        if (!prevProps.network.isOffline || this.props.network.isOffline) {
+        if (!prevProps.network.isOffline || this.props.network.isOffline || this.hasPlaidBankAccountsData()) {
             return;
         }
 
-        // If we are coming back from offline, we need to re-run our call to kick off Plaid
+        // If we are coming back from offline and we haven't had bank accounts data yet, we need to re-run our call to kick off Plaid
         BankAccounts.openPlaidBankLogin(this.props.allowDebit, this.props.bankAccountID);
     }
 
@@ -109,6 +107,13 @@ class AddPlaidBankAccount extends React.Component {
         }
     }
 
+    /**
+     * @returns {Boolean}
+     */
+    hasPlaidBankAccountsData() {
+        return !_.isEmpty(lodashGet(this.props.plaidData, 'bankAccounts')) || !_.isEmpty(lodashGet(this.props.plaidData, 'errors'));
+    }
+
     render() {
         const plaidBankAccounts = lodashGet(this.props.plaidData, 'bankAccounts') || [];
         const token = this.getPlaidLinkToken();