From 55074f391d757c0c6f7a1761b3327c3bac3be643 Mon Sep 17 00:00:00 2001
From: Shekar Siri <sshekarsiri@gmail.com>
Date: Wed, 5 May 2021 17:16:02 +0530
Subject: [PATCH] sign up check for tenants

---
 frontend/app/Router.js                        | 12 ++++++---
 frontend/app/components/Header/Header.js      |  2 +-
 .../Signup/SignupForm/SignupForm.js           | 27 +++++--------------
 frontend/app/duck/user.js                     |  1 +
 4 files changed, 16 insertions(+), 26 deletions(-)

diff --git a/frontend/app/Router.js b/frontend/app/Router.js
index 3bdfc3c8f..4885aa27f 100644
--- a/frontend/app/Router.js
+++ b/frontend/app/Router.js
@@ -22,6 +22,7 @@ import FunnelIssueDetails from 'Components/Funnels/FunnelIssueDetails';
 import APIClient from './api_client';
 import * as routes from './routes';
 import Signup from './components/Signup/Signup';
+import { fetchTenants } from 'Duck/user';
 
 const BugFinder = withSiteIdUpdater(BugFinderPure);
 const Dashboard = withSiteIdUpdater(DashboardPure);
@@ -54,20 +55,22 @@ const ONBOARDING_PATH = routes.onboarding();
   const jwt = state.get('jwt');
   const changePassword = state.getIn([ 'user', 'account', 'changePassword' ]);
   const userInfoLoading = state.getIn([ 'user', 'fetchUserInfoRequest', 'loading' ]);
+  const fetchingTenants = state.get('user', 'fetchTenantsRequest', 'loading');
   return {
     jwt,
     siteId,
     changePassword,
     sites: state.getIn([ 'user', 'client', 'sites' ]),
     isLoggedIn: jwt !== null && !changePassword,
-    loading: siteId === null || userInfoLoading,
+    loading: siteId === null || userInfoLoading || fetchingTenants,
     email: state.getIn([ 'user', 'account', 'email' ]),
     account: state.getIn([ 'user', 'account' ]),
     organisation: state.getIn([ 'user', 'client', 'name' ]),
     tenantId: state.getIn([ 'user', 'client', 'tenantId' ]),
+    tenants: state.getIn(['user', 'tenants']),
   };
 }, {
-  fetchUserInfo,
+  fetchUserInfo, fetchTenants
 })
 class Router extends React.Component {
   constructor(props) {
@@ -76,6 +79,7 @@ class Router extends React.Component {
       Promise.all([props.fetchUserInfo()])
       .then(() => this.onLoginLogout());
     }
+    props.fetchTenants();
   }
 
   componentDidUpdate(prevProps) {
@@ -89,7 +93,7 @@ class Router extends React.Component {
   }
 
   render() {    
-    const { isLoggedIn, jwt, siteId, sites, loading, changePassword, location } = this.props;
+    const { isLoggedIn, jwt, siteId, sites, loading, changePassword, location, tenants } = this.props;
     const siteIdList = sites.map(({ id }) => id).toJS();
     const hideHeader = location.pathname && location.pathname.includes('/session/');
 
@@ -138,7 +142,7 @@ class Router extends React.Component {
       <Switch>
         <Route exact strict path={ FORGOT_PASSWORD } component={ ForgotPassword } />
         <Route exact strict path={ LOGIN_PATH } component={ changePassword ? UpdatePassword : Login } />
-        <Route exact strict path={ SIGNUP_PATH } component={ Signup } />
+        { tenants.length === 0 && <Route exact strict path={ SIGNUP_PATH } component={ Signup } /> }
         <Redirect to={ LOGIN_PATH } />
       </Switch>;
   }
diff --git a/frontend/app/components/Header/Header.js b/frontend/app/components/Header/Header.js
index 6e7a6601f..fc3fb870a 100644
--- a/frontend/app/components/Header/Header.js
+++ b/frontend/app/components/Header/Header.js
@@ -99,7 +99,7 @@ const Header = (props) => {
         to={ withSiteId(DASHBOARD_PATH, siteId) }
         className={ styles.nav }
         activeClassName={ styles.active }
-      >                
+      >         
         <span>{ 'Metrics' }</span>
       </NavLink>
       <div className={ styles.right }>
diff --git a/frontend/app/components/Signup/SignupForm/SignupForm.js b/frontend/app/components/Signup/SignupForm/SignupForm.js
index b49750016..2bfd480bc 100644
--- a/frontend/app/components/Signup/SignupForm/SignupForm.js
+++ b/frontend/app/components/Signup/SignupForm/SignupForm.js
@@ -1,13 +1,11 @@
-import React, { useState } from 'react'
-import { Icon, Loader, Button, Link, Dropdown, CircularLoader } from 'UI'
-import { forgotPassword, login } from 'App/routes'
+import React from 'react'
+import { Icon, Button, Link, Dropdown, CircularLoader } from 'UI'
+import { login } from 'App/routes'
 import ReCAPTCHA from 'react-google-recaptcha'
 import stl from './signup.css'
-import cn from 'classnames'
-import { signup, fetchTenants } from 'Duck/user';
+import { signup } from 'Duck/user';
 import { connect } from 'react-redux'
 
-const FORGOT_PASSWORD = forgotPassword()
 const LOGIN_ROUTE = login()
 const recaptchaRef = React.createRef()
 
@@ -17,7 +15,7 @@ const recaptchaRef = React.createRef()
     errors: state.getIn([ 'user', 'signupRequest', 'errors' ]),
     loading: state.getIn([ 'user', 'signupRequest', 'loading' ]),
   }),
-  { signup, fetchTenants },
+  { signup },
 )
 export default class SignupForm extends React.Component {
 
@@ -30,10 +28,6 @@ export default class SignupForm extends React.Component {
     organizationName: '',
   };
 
-  componentDidMount() {
-    this.props.fetchTenants();
-  }
-
   handleSubmit = (token) => {
     const { tenantId, fullname, password, email, projectName, organizationName, auth } = this.state;
     this.props.signup({ tenantId, fullname, password, email, projectName, organizationName, auth, 'g-recaptcha-response': token })
@@ -165,17 +159,8 @@ export default class SignupForm extends React.Component {
               'Signup' 
             }
           </Button>
-  
-          {/* <div className={ cn(stl.links, 'text-lg') }>
-            <Link to={ LOGIN_ROUTE }>{'Back to Login'}</Link>
-          </div> */}
         </div>
       </form>
     )
   }
-}
-
-// export default connect(state => ({
-//   errors: state.getIn([ 'user', 'signupRequest', 'errors' ]),
-//   loading: state.getIn([ 'user', 'signupRequest', 'loading' ]),
-// }), { signup })(SignupForm)
\ No newline at end of file
+}
\ No newline at end of file
diff --git a/frontend/app/duck/user.js b/frontend/app/duck/user.js
index 83430e72d..3dad02a10 100644
--- a/frontend/app/duck/user.js
+++ b/frontend/app/duck/user.js
@@ -91,6 +91,7 @@ export default withRequestState({
   putClientRequest: PUT_CLIENT,
   updateAccountRequest: UPDATE_ACCOUNT,
   updateAppearance: UPDATE_APPEARANCE,
+  fetchTenantsRequest: FETCH_TENANTS,
 }, reducer);
 
 export const login = params => dispatch => dispatch({