From faeb4c46d18b8a890a67e9522cfd3f0d15015d90 Mon Sep 17 00:00:00 2001
From: evpozdniakov <evpozdniakov@gmail.com>
Date: Tue, 24 Oct 2017 22:25:21 -0400
Subject: [PATCH 1/6] Apply firebase config + few changes of previous homework.

---
 public/index.html                       | 17 ++++++++++
 src/components/App.js                   | 45 ++++++++++++++++++++++---
 src/components/common/ErrorField.js     |  4 +--
 src/components/common/ProtectedRoute.js |  4 +--
 src/components/common/Submenu.js        | 30 +++++++++++++++++
 src/components/routes/Auth.js           |  8 ++---
 src/config.js                           |  6 ++--
 src/ducks/auth.js                       | 10 ++++++
 8 files changed, 107 insertions(+), 17 deletions(-)
 create mode 100644 src/components/common/Submenu.js
diff --git a/public/index.html b/public/index.html
index 7bee027..6ffdc56 100644
--- a/public/index.html
+++ b/public/index.html
@@ -20,6 +20,23 @@
       Learn how to configure a non-root public URL by running `npm run build`.
     -->
     <title>React App</title>
+    <style>
+      nav.submenu {
+        background: #eee;
+        padding: .5rem;
+      }
+      nav.submenu a {
+        margin-right: 1rem;
+        color: #333;
+        text-decoration: none;
+      }
+      nav.submenu a.active {
+        font-weight: bold;
+      }
+      .error-field {
+        margin-bottom: 1rem;
+      }
+    </style>
   </head>
   <body>
     <noscript>
diff --git a/src/components/App.js b/src/components/App.js
index af31092..eeff2b9 100644
--- a/src/components/App.js
+++ b/src/components/App.js
@@ -1,9 +1,13 @@
 import React, { Component } from 'react'
 import {Route, NavLink} from 'react-router-dom'
+import {connect} from 'react-redux'
+import firebase from 'firebase'
 import ProtectedRoute from './common/ProtectedRoute'
 import AdminPage from './routes/Admin'
 import AuthPage from './routes/Auth'
 import PersonPage from './routes/PersonPage'
+import Submenu from './common/Submenu'
+import {userAuthorized} from '../ducks/auth'
 
 class App extends Component {
     static propTypes = {
@@ -14,16 +18,47 @@ class App extends Component {
         return (
             <div>
                 <h1>Hello world</h1>
-                <ul>
-                    <li><NavLink to='/admin' activeStyle = {{color: 'red'}}>admin</NavLink></li>
-                    <li><NavLink to='/people' activeStyle = {{color: 'red'}}>people</NavLink></li>
-                </ul>
+                {this.renderSubmenu()}
                 <ProtectedRoute path = '/admin' component = {AdminPage}/>
                 <ProtectedRoute path = '/people' component={PersonPage}/>
                 <Route path = '/auth' component = {AuthPage}/>
             </div>
         )
     }
+
+    renderSubmenu() {
+        const links = []
+
+        if (this.props.authorized) {
+            links.push({
+                to: '/admin',
+                title: 'Admin',
+            }, {
+                to: '/people',
+                title: 'People',
+            }, {
+                to: '/logout',
+                title: 'Logout',
+            })
+        }
+        else {
+            links.push({
+                to: '/auth/signin',
+                title: 'Sign in',
+            }, {
+                to: '/auth/signup',
+                title: 'Sign up',
+            })
+        }
+
+        const props = {
+            links,
+        }
+
+        return <Submenu {...props} />
+    }
 }
 
-export default App
\ No newline at end of file
+export default connect(state => ({
+    authorized: userAuthorized(state)
+}), null, null, { pure: false })(App)
diff --git a/src/components/common/ErrorField.js b/src/components/common/ErrorField.js
index 925999d..3accb2d 100644
--- a/src/components/common/ErrorField.js
+++ b/src/components/common/ErrorField.js
@@ -4,8 +4,8 @@ function ErrorField(props) {
     const {input, type, meta: {error, touched}} = props
     const errorText = touched && error && <div style={{color: 'red'}}>{error}</div>
     return (
-        <div>
-            <label>{input.name}</label>
+        <div className="error-field">
+            <label>{input.name}<br/></label>
             <input {...input} type={type}/>
             {errorText}
         </div>
diff --git a/src/components/common/ProtectedRoute.js b/src/components/common/ProtectedRoute.js
index da53475..fe4e302 100644
--- a/src/components/common/ProtectedRoute.js
+++ b/src/components/common/ProtectedRoute.js
@@ -1,7 +1,7 @@
 import React, { Component } from 'react'
 import {Route} from 'react-router-dom'
 import {connect} from 'react-redux'
-import {userSelector} from '../../ducks/auth'
+import {userAuthorized} from '../../ducks/auth'
 
 class ProtectedRoute extends Component {
     static propTypes = {
@@ -21,5 +21,5 @@ class ProtectedRoute extends Component {
 }
 
 export default connect(state => ({
-    authorized: !!userSelector(state)
+    authorized: userAuthorized(state)
 }), null, null, { pure: false })(ProtectedRoute)
\ No newline at end of file
diff --git a/src/components/common/Submenu.js b/src/components/common/Submenu.js
new file mode 100644
index 0000000..370f97e
--- /dev/null
+++ b/src/components/common/Submenu.js
@@ -0,0 +1,30 @@
+import React, { Component } from 'react'
+import { NavLink } from 'react-router-dom'
+
+class Submenu extends Component {
+  render() {
+    return (
+      <nav className="submenu">
+        {this.props.links.map(this.renderSubmenuItem.bind(this))}
+      </nav>
+    )
+  }
+
+  renderSubmenuItem(item, index) {
+    const { to, title } = item
+
+    const props = {
+      key: index,
+      activeClassName: 'active',
+      to,
+    }
+
+    return (
+      <NavLink {...props}>
+        {title}
+      </NavLink>
+    )
+  }
+}
+
+export default Submenu
diff --git a/src/components/routes/Auth.js b/src/components/routes/Auth.js
index a183ff9..5c587bc 100644
--- a/src/components/routes/Auth.js
+++ b/src/components/routes/Auth.js
@@ -1,7 +1,7 @@
 import React, { Component } from 'react'
 import {Route, NavLink} from 'react-router-dom'
 import {connect} from 'react-redux'
-import {signUp} from '../../ducks/auth'
+import {signUp, signIn} from '../../ducks/auth'
 import SignIn from '../auth/SignIn'
 import SignUp from '../auth/SignUp'
 
@@ -14,16 +14,14 @@ class AuthPage extends Component {
         return (
             <div>
                 <h2>Auth page</h2>
-                <NavLink to = '/auth/signin' activeStyle = {{color: 'red'}}>sign in</NavLink>
-                <NavLink to = '/auth/signup' activeStyle = {{color: 'red'}}>sign up</NavLink>
                 <Route path = '/auth/signin' render = {() => <SignIn onSubmit = {this.handleSignIn}/>}/>
                 <Route path = '/auth/signup' render = {() => <SignUp onSubmit = {this.handleSignUp}/>}/>
             </div>
         )
     }
 
-    handleSignIn = (values) => console.log('---', 'sign in', values)
+    handleSignIn = ({email, password}) => this.props.signIn(email, password)
     handleSignUp = ({email, password}) => this.props.signUp(email, password)
 }
 
-export default connect(null, { signUp })(AuthPage)
\ No newline at end of file
+export default connect(null, { signIn, signUp })(AuthPage)
diff --git a/src/config.js b/src/config.js
index 2782a0b..6ce5a80 100644
--- a/src/config.js
+++ b/src/config.js
@@ -1,12 +1,12 @@
 import firebase from 'firebase'
 
-export const appName = 'advreact-1610'
+export const appName = 'advreact-1610-9cb0c'
 
 firebase.initializeApp({
-    apiKey: "AIzaSyB31xpTtp4Jln_hb2kAbE4PGf6Mi8EgLyA",
+    apiKey: "AIzaSyBNaOQkLL75ZGoeaNtqbe63wEjjWLzLOPY",
     authDomain: `${appName}.firebaseapp.com`,
     databaseURL: `https://${appName}.firebaseio.com`,
     projectId: appName,
     storageBucket: "",
-    messagingSenderId: "397157634637"
+    messagingSenderId: "266748171955"
 })
\ No newline at end of file
diff --git a/src/ducks/auth.js b/src/ducks/auth.js
index 25f3bf7..7b21fa3 100644
--- a/src/ducks/auth.js
+++ b/src/ducks/auth.js
@@ -10,6 +10,7 @@ import {call, put, all, take} from 'redux-saga/effects'
 export const moduleName = 'auth'
 const prefix = `${appName}/${moduleName}`
 
+export const SIGN_IN_START = `${prefix}/SIGN_IN_START`
 export const SIGN_UP_START = `${prefix}/SIGN_UP_START`
 export const SIGN_UP_SUCCESS = `${prefix}/SIGN_UP_SUCCESS`
 export const SIGN_UP_ERROR = `${prefix}/SIGN_UP_ERROR`
@@ -30,6 +31,7 @@ export default function reducer(state = new ReducerRecord(), action) {
 
     switch (type) {
         case SIGN_UP_START:
+        case SIGN_IN_START:
             return state.set('loading', true)
 
         case SIGN_UP_SUCCESS:
@@ -49,6 +51,7 @@ export default function reducer(state = new ReducerRecord(), action) {
 
 export const stateSelector = state => state[moduleName]
 export const userSelector = createSelector(stateSelector, state => state.user)
+export const userAuthorized = createSelector(stateSelector, state => !!state.user)
 
 /**
  * Action Creators
@@ -60,6 +63,13 @@ export function signUp(email, password) {
     }
 }
 
+export function signIn(email, password) {
+    return {
+        type: SIGN_IN_START,
+        payload: { email, password }
+    }
+}
+
 firebase.auth().onAuthStateChanged(user => {
     if (!user) return
 

From 061fe7c36301842794ec74ebd0d05009d23eb686 Mon Sep 17 00:00:00 2001
From: evpozdniakov <evpozdniakov@gmail.com>
Date: Tue, 24 Oct 2017 23:36:33 -0400
Subject: [PATCH 2/6] HT2.1: sign-in saga applied; errorMessage selector added
 to display sign in error.

---
 public/index.html             |  3 +++
 src/components/auth/SignIn.js | 13 ++++++++++-
 src/components/routes/Auth.js | 17 +++++++++++---
 src/ducks/auth.js             | 44 +++++++++++++++++++++++++++++++----
 4 files changed, 69 insertions(+), 8 deletions(-)

diff --git a/public/index.html b/public/index.html
index 6ffdc56..88b2cd3 100644
--- a/public/index.html
+++ b/public/index.html
@@ -36,6 +36,9 @@
       .error-field {
         margin-bottom: 1rem;
       }
+      .error {
+        color: #c00;
+      }
     </style>
   </head>
   <body>
diff --git a/src/components/auth/SignIn.js b/src/components/auth/SignIn.js
index 57e09b9..c8da2a9 100644
--- a/src/components/auth/SignIn.js
+++ b/src/components/auth/SignIn.js
@@ -20,12 +20,23 @@ class SignIn extends Component {
                     <div>
                         <input type = 'submit'/>
                     </div>
+                    {this.renderError()}
                 </form>
             </div>
         )
     }
+
+    renderError() {
+        const { signInError } = this.props
+
+        if (!signInError) {
+            return null
+        }
+
+        return <p className="error">{signInError}</p>
+    }
 }
 
 export default reduxForm({
     form: 'auth'
-})(SignIn)
\ No newline at end of file
+})(SignIn)
diff --git a/src/components/routes/Auth.js b/src/components/routes/Auth.js
index 5c587bc..ebe6ab9 100644
--- a/src/components/routes/Auth.js
+++ b/src/components/routes/Auth.js
@@ -1,7 +1,7 @@
 import React, { Component } from 'react'
 import {Route, NavLink} from 'react-router-dom'
 import {connect} from 'react-redux'
-import {signUp, signIn} from '../../ducks/auth'
+import {signUp, signIn, errorMessage} from '../../ducks/auth'
 import SignIn from '../auth/SignIn'
 import SignUp from '../auth/SignUp'
 
@@ -14,14 +14,25 @@ class AuthPage extends Component {
         return (
             <div>
                 <h2>Auth page</h2>
-                <Route path = '/auth/signin' render = {() => <SignIn onSubmit = {this.handleSignIn}/>}/>
+                <Route path = '/auth/signin' render = {this.renderSignIn.bind(this)}/>
                 <Route path = '/auth/signup' render = {() => <SignUp onSubmit = {this.handleSignUp}/>}/>
             </div>
         )
     }
 
+    renderSignIn() {
+        const props = {
+            onSubmit: this.handleSignIn,
+            signInError: this.props.error,
+        }
+
+        return <SignIn {...props} />
+    }
+
     handleSignIn = ({email, password}) => this.props.signIn(email, password)
     handleSignUp = ({email, password}) => this.props.signUp(email, password)
 }
 
-export default connect(null, { signIn, signUp })(AuthPage)
+export default connect(state => ({
+    error: errorMessage(state)
+}), { signIn, signUp, errorMessage })(AuthPage)
diff --git a/src/ducks/auth.js b/src/ducks/auth.js
index 7b21fa3..027d84d 100644
--- a/src/ducks/auth.js
+++ b/src/ducks/auth.js
@@ -10,12 +10,14 @@ import {call, put, all, take} from 'redux-saga/effects'
 export const moduleName = 'auth'
 const prefix = `${appName}/${moduleName}`
 
-export const SIGN_IN_START = `${prefix}/SIGN_IN_START`
 export const SIGN_UP_START = `${prefix}/SIGN_UP_START`
 export const SIGN_UP_SUCCESS = `${prefix}/SIGN_UP_SUCCESS`
 export const SIGN_UP_ERROR = `${prefix}/SIGN_UP_ERROR`
 
+export const SIGN_IN_START = `${prefix}/SIGN_IN_START`
 export const SIGN_IN_SUCCESS = `${prefix}/SIGN_IN_SUCCESS`
+export const SIGN_IN_ERROR = `${prefix}/SIGN_IN_ERROR`
+
 
 /**
  * Reducer
@@ -32,7 +34,9 @@ export default function reducer(state = new ReducerRecord(), action) {
     switch (type) {
         case SIGN_UP_START:
         case SIGN_IN_START:
-            return state.set('loading', true)
+            return state
+                .set('loading', true)
+                .set('error', null)
 
         case SIGN_UP_SUCCESS:
         case SIGN_IN_SUCCESS:
@@ -40,6 +44,12 @@ export default function reducer(state = new ReducerRecord(), action) {
                 .set('user', payload.user)
                 .set('loading', false)
 
+        case SIGN_UP_ERROR:
+        case SIGN_IN_ERROR:
+            return state
+                .set('error', payload.error)
+                .set('loading', false)
+
         default:
             return state
     }
@@ -52,6 +62,7 @@ export default function reducer(state = new ReducerRecord(), action) {
 export const stateSelector = state => state[moduleName]
 export const userSelector = createSelector(stateSelector, state => state.user)
 export const userAuthorized = createSelector(stateSelector, state => !!state.user)
+export const errorMessage = createSelector(stateSelector, state => state.error && state.error.message || '')
 
 /**
  * Action Creators
@@ -106,8 +117,33 @@ export function * signUpSaga() {
     }
 }
 
+export function * signInSaga() {
+    const auth = firebase.auth()
+
+    while (true) {
+        const action = yield take(SIGN_IN_START)
+        const { email, password } = action.payload
+
+        try {
+            const user = yield call([auth, auth.signInWithEmailAndPassword], email, password)
+
+            yield put({
+                type: SIGN_IN_SUCCESS,
+                payload: {user},
+            })
+        }
+        catch (error) {
+            yield put({
+                type: SIGN_IN_ERROR,
+                payload: {error},
+            })
+        }
+    }
+}
+
 export function * saga() {
     yield all([
-        signUpSaga()
+        signUpSaga(),
+        signInSaga(),
     ])
-}
\ No newline at end of file
+}

From 86e8bf25ac149874044c01688562b4c0dafc72d4 Mon Sep 17 00:00:00 2001
From: evpozdniakov <evpozdniakov@gmail.com>
Date: Wed, 25 Oct 2017 01:14:14 -0400
Subject: [PATCH 3/6] Test singInSaga

---
 src/ducks/auth.test.js | 60 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 60 insertions(+)
 create mode 100644 src/ducks/auth.test.js

diff --git a/src/ducks/auth.test.js b/src/ducks/auth.test.js
new file mode 100644
index 0000000..4f1ba4d
--- /dev/null
+++ b/src/ducks/auth.test.js
@@ -0,0 +1,60 @@
+import firebase from 'firebase'
+import {call, put, take} from 'redux-saga/effects'
+import { 
+    SIGN_IN_START,
+    SIGN_IN_SUCCESS,
+    SIGN_IN_ERROR,
+    signIn,
+    signUpSaga,
+    signInSaga,
+} from './auth'
+
+describe('Sagas', () => {
+    test('signInSaga success', () => {
+        expect.assertions(4);
+
+        const auth = firebase.auth()
+        const email = 'email'
+        const password = 'pswd'
+        const payload = {email, password}
+        const request = signIn(payload)
+        const gen = signInSaga(request)
+        const user = {the: 'user'}
+
+        expect(gen.next().value).toEqual(take(SIGN_IN_START))
+
+        expect(gen.next({payload}).value).toEqual(call(
+            [auth, auth.signInWithEmailAndPassword], email, password))
+
+        expect(gen.next(user).value).toEqual(put({
+            type: SIGN_IN_SUCCESS,
+            payload: {user},
+        }))
+
+        expect(gen.next().value).toEqual(take(SIGN_IN_START))
+    })
+
+    test('signInSaga fail', () => {
+        expect.assertions(4);
+
+        const auth = firebase.auth()
+        const email = 'email'
+        const password = 'pswd'
+        const payload = {email, password}
+        const request = signIn(payload)
+        const gen = signInSaga(request)
+        const error = new Error('fail')
+
+        expect(gen.next().value).toEqual(take(SIGN_IN_START))
+
+        expect(gen.next({payload}).value).toEqual(call(
+            [auth, auth.signInWithEmailAndPassword], email, password))
+
+        expect(gen.throw(error).value).toEqual(put({
+            type: SIGN_IN_ERROR,
+            payload: {error},
+        }))
+
+        expect(gen.next().value).toEqual(take(SIGN_IN_START))
+    })
+})

From 12318ba7fc053b2f39114deee8f58d37130c4181 Mon Sep 17 00:00:00 2001
From: evpozdniakov <evpozdniakov@gmail.com>
Date: Wed, 25 Oct 2017 09:38:05 -0400
Subject: [PATCH 4/6] Test singUpSaga; attempt to test combining saga

---
 src/ducks/auth.test.js | 78 +++++++++++++++++++++++++++++++++++++-----
 1 file changed, 69 insertions(+), 9 deletions(-)

diff --git a/src/ducks/auth.test.js b/src/ducks/auth.test.js
index 4f1ba4d..0490d39 100644
--- a/src/ducks/auth.test.js
+++ b/src/ducks/auth.test.js
@@ -1,15 +1,75 @@
 import firebase from 'firebase'
-import {call, put, take} from 'redux-saga/effects'
-import { 
-    SIGN_IN_START,
-    SIGN_IN_SUCCESS,
-    SIGN_IN_ERROR,
-    signIn,
-    signUpSaga,
-    signInSaga,
+import {all, call, put, take} from 'redux-saga/effects'
+import {
+    SIGN_UP_START, SIGN_UP_SUCCESS, SIGN_UP_ERROR,
+    SIGN_IN_START, SIGN_IN_SUCCESS, SIGN_IN_ERROR,
+    signUp, signIn,
+    saga, signUpSaga, signInSaga,
 } from './auth'
 
 describe('Sagas', () => {
+    test('combining saga', () => {
+        const gen = saga()
+
+        const { value } = gen.next()
+
+        // this one fails. how to make it work? TY
+        /*expect(value).toEqual(all([
+            signUpSaga(),
+            signInSaga(),
+        ]))*/
+
+        expect(gen.next().done).toBeTruthy()
+    })
+
+    test('signUpSaga success', () => {
+        expect.assertions(4)
+
+        const auth = firebase.auth()
+        const email = 'email'
+        const password = 'pswd'
+        const user = {the: 'user'}
+        const payload = {email, password}
+        const request = signUp(email, password)
+        const gen = signUpSaga(request)
+
+        expect(gen.next().value).toEqual(take(SIGN_UP_START))
+
+        expect(gen.next({payload}).value).toEqual(call(
+            [auth, auth.createUserWithEmailAndPassword], email, password))
+
+        expect(gen.next(user).value).toEqual(put({
+            type: SIGN_UP_SUCCESS,
+            payload: {user}
+        }))
+
+        expect(gen.next().value).toEqual(take(SIGN_UP_START))
+    })
+
+    test('signUpSaga error', () => {
+        expect.assertions(4)
+
+        const auth = firebase.auth()
+        const email = 'email'
+        const password = 'pswd'
+        const error = new Error('fail')
+        const payload = {email, password}
+        const request = signUp(email, password)
+        const gen = signUpSaga(request)
+
+        expect(gen.next().value).toEqual(take(SIGN_UP_START))
+
+        expect(gen.next({payload}).value).toEqual(call(
+            [auth, auth.createUserWithEmailAndPassword], email, password))
+
+        expect(gen.throw(error).value).toEqual(put({
+            type: SIGN_UP_ERROR,
+            payload: {error}
+        }))
+
+        expect(gen.next().value).toEqual(take(SIGN_UP_START))
+    })
+
     test('signInSaga success', () => {
         expect.assertions(4);
 
@@ -34,7 +94,7 @@ describe('Sagas', () => {
         expect(gen.next().value).toEqual(take(SIGN_IN_START))
     })
 
-    test('signInSaga fail', () => {
+    test('signInSaga error', () => {
         expect.assertions(4);
 
         const auth = firebase.auth()

From bd00972cdde83bb43444e8efb225c3ba91220a21 Mon Sep 17 00:00:00 2001
From: evpozdniakov <evpozdniakov@gmail.com>
Date: Wed, 25 Oct 2017 10:19:08 -0400
Subject: [PATCH 5/6] HT2.3: redirect to /people after successful sign in;
 update tests.

---
 src/components/App.js  | 5 +++++
 src/ducks/auth.js      | 3 +++
 src/ducks/auth.test.js | 6 +++++-
 3 files changed, 13 insertions(+), 1 deletion(-)

diff --git a/src/components/App.js b/src/components/App.js
index eeff2b9..81f6e17 100644
--- a/src/components/App.js
+++ b/src/components/App.js
@@ -8,12 +8,17 @@ import AuthPage from './routes/Auth'
 import PersonPage from './routes/PersonPage'
 import Submenu from './common/Submenu'
 import {userAuthorized} from '../ducks/auth'
+import history from '../history'
 
 class App extends Component {
     static propTypes = {
 
     };
 
+    componentDidMount() {
+        history.push(this.props.authorized ? '/people' : '/auth/signin')
+    }
+
     render() {
         return (
             <div>
diff --git a/src/ducks/auth.js b/src/ducks/auth.js
index 027d84d..cb6a563 100644
--- a/src/ducks/auth.js
+++ b/src/ducks/auth.js
@@ -3,6 +3,7 @@ import {Record} from 'immutable'
 import firebase from 'firebase'
 import {createSelector} from 'reselect'
 import {call, put, all, take} from 'redux-saga/effects'
+import history from '../history'
 
 /**
  * Constants
@@ -131,6 +132,8 @@ export function * signInSaga() {
                 type: SIGN_IN_SUCCESS,
                 payload: {user},
             })
+
+            yield call([history, history.push], '/people')
         }
         catch (error) {
             yield put({
diff --git a/src/ducks/auth.test.js b/src/ducks/auth.test.js
index 0490d39..3f22b0c 100644
--- a/src/ducks/auth.test.js
+++ b/src/ducks/auth.test.js
@@ -1,4 +1,5 @@
 import firebase from 'firebase'
+import history from '../history'
 import {all, call, put, take} from 'redux-saga/effects'
 import {
     SIGN_UP_START, SIGN_UP_SUCCESS, SIGN_UP_ERROR,
@@ -71,7 +72,7 @@ describe('Sagas', () => {
     })
 
     test('signInSaga success', () => {
-        expect.assertions(4);
+        expect.assertions(5);
 
         const auth = firebase.auth()
         const email = 'email'
@@ -91,6 +92,9 @@ describe('Sagas', () => {
             payload: {user},
         }))
 
+        expect(gen.next().value).toEqual(call(
+            [history, history.push], '/people'))
+
         expect(gen.next().value).toEqual(take(SIGN_IN_START))
     })
 

From 65e1ed2e2062160bf6e7511c37c40f9f2fff84ed Mon Sep 17 00:00:00 2001
From: evpozdniakov <evpozdniakov@gmail.com>
Date: Wed, 25 Oct 2017 10:52:41 -0400
Subject: [PATCH 6/6] HT2.4: reset form after successful sign in (bad way)

---
 src/components/auth/SignIn.js | 9 +++++++++
 src/components/routes/Auth.js | 6 ++++--
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/src/components/auth/SignIn.js b/src/components/auth/SignIn.js
index c8da2a9..e6e0647 100644
--- a/src/components/auth/SignIn.js
+++ b/src/components/auth/SignIn.js
@@ -6,6 +6,15 @@ class SignIn extends Component {
 
     };
 
+    shouldComponentUpdate(nextProps) {
+        if (nextProps.shouldReset) {
+            this.props.reset()
+            return false
+        }
+
+        return true
+    }
+
     render() {
         return (
             <div>
diff --git a/src/components/routes/Auth.js b/src/components/routes/Auth.js
index ebe6ab9..d06aa85 100644
--- a/src/components/routes/Auth.js
+++ b/src/components/routes/Auth.js
@@ -1,7 +1,7 @@
 import React, { Component } from 'react'
 import {Route, NavLink} from 'react-router-dom'
 import {connect} from 'react-redux'
-import {signUp, signIn, errorMessage} from '../../ducks/auth'
+import {signUp, signIn, errorMessage, userAuthorized} from '../../ducks/auth'
 import SignIn from '../auth/SignIn'
 import SignUp from '../auth/SignUp'
 
@@ -24,6 +24,7 @@ class AuthPage extends Component {
         const props = {
             onSubmit: this.handleSignIn,
             signInError: this.props.error,
+            shouldReset: this.props.authorized,
         }
 
         return <SignIn {...props} />
@@ -34,5 +35,6 @@ class AuthPage extends Component {
 }
 
 export default connect(state => ({
-    error: errorMessage(state)
+    error: errorMessage(state),
+    authorized: userAuthorized(state),
 }), { signIn, signUp, errorMessage })(AuthPage)
