Merge pull request #14 from paulkevany/auth-loading

Fix auth loading spinner bug

frontend/.gitignore

@@ -6,7 +6,6 @@ jspm_packages
 .serverless
 
 #amplify
-/amplify
 amplify/\#current-cloud-backend
 amplify/.config/local-*
 amplify/mock-data
@@ -19,4 +18,4 @@ aws-exports.js
 awsconfiguration.json
 amplifyconfiguration.json
 amplify-gradle-config.json
-amplifyxc.config
+amplifyxc.config

frontend/src/Components/Login.js

@@ -53,7 +53,7 @@ class Login extends Component {
   handleChange = ({ target: { id, value } }) => this.setState({ [id]: value })
 
   validateInput = () =>
-    this.state.email.length > 0 && this.state.password.length > 0
+    this.state.email.length > 3 && this.state.password.length >= 8
 
   handleSubmit = event => {
     event.preventDefault()
@@ -106,6 +106,7 @@ class Login extends Component {
                 variant="contained"
                 className={classes.button}
                 color="primary"
+                disabled={loggingIn || !this.validateInput()}
               >
                 Login
               </Button>

frontend/src/Reducers/auth.js

@@ -1,4 +1,10 @@
-import { LOGIN_REQUEST, LOGIN_SUCCESS, LOGIN_FAILED } from '../Actions/auth'
+import {
+  LOGIN_REQUEST,
+  LOGIN_SUCCESS,
+  LOGIN_FAILED,
+  AUTHENTICATION_VALIDATED,
+  AUTHENTICATION_INVALIDATED
+} from '../Actions/auth'
 
 const defaultState = {
   loggingIn: false,
@@ -16,13 +22,20 @@ export default (state = defaultState, { type, meta, payload, error }) => {
       }
 
     case LOGIN_SUCCESS:
+    case AUTHENTICATION_VALIDATED:
       return {
         ...state,
         loggingIn: false,
         authenticated: true,
         loginError: null
       }
 
+    case AUTHENTICATION_INVALIDATED:
+      return {
+        ...state,
+        authenticated: false
+      }
+
     case LOGIN_FAILED:
       return {
         ...state,