From 684646c78566afc0c4437f66780b5fb6158c19cd Mon Sep 17 00:00:00 2001
From: catarak <ctarakajian@gmail.com>
Date: Mon, 27 Jun 2016 16:14:26 -0400
Subject: [PATCH] add proptypes to nav, fix proptypes for login

---
 client/components/Nav.js                    | 71 +++++++++++----------
 client/modules/App/App.js                   |  6 +-
 client/modules/User/components/LoginForm.js |  2 +-
 3 files changed, 45 insertions(+), 34 deletions(-)

diff --git a/client/components/Nav.js b/client/components/Nav.js
index f1e931a6..687f4b7a 100644
--- a/client/components/Nav.js
+++ b/client/components/Nav.js
@@ -1,37 +1,44 @@
-import React from 'react';
+import React, { PropTypes } from 'react';
 import { Link } from 'react-router';
 
-class Nav extends React.Component {
-  render() {
-    return (
-      <nav className="nav">
-        <ul className="nav__items-left">
-          <li className="nav__item">
-            <p
-              className="nav__new"
-              onClick={this.props.createProject}
-            >
-              New
-            </p>
-          </li>
-          <li className="nav__item">
-            <p
-              className="nav__save"
-              onClick={this.props.saveProject}
-            >
-              Save
-            </p>
-          </li>
-        </ul>
-        <ul className="nav__items-right">
-          <li className="nav__item">
-            {this.props.user.authenticated && <p>Hello, {this.props.user.username}!</p>}
-            {!this.props.user.authenticated && <p><Link to="/login">Login</Link> or <Link to="/signup">Sign Up</Link></p>}
-          </li>
-        </ul>
-      </nav>
-    );
-  }
+function Nav(props) {
+  return (
+    <nav className="nav">
+      <ul className="nav__items-left">
+        <li className="nav__item">
+          <p
+            className="nav__new"
+            onClick={props.createProject}
+          >
+            New
+          </p>
+        </li>
+        <li className="nav__item">
+          <p
+            className="nav__save"
+            onClick={props.saveProject}
+          >
+            Save
+          </p>
+        </li>
+      </ul>
+      <ul className="nav__items-right">
+        <li className="nav__item">
+          {props.user.authenticated && <p>Hello, {props.user.username}!</p>}
+          {!props.user.authenticated && <p><Link to="/login">Login</Link> or <Link to="/signup">Sign Up</Link></p>}
+        </li>
+      </ul>
+    </nav>
+  );
 }
 
+Nav.propTypes = {
+  createProject: PropTypes.func.isRequired,
+  saveProject: PropTypes.func.isRequired,
+  user: PropTypes.shape({
+    authenticated: PropTypes.bool.isRequired,
+    username: PropTypes.string
+  }).isRequired
+};
+
 export default Nav;
diff --git a/client/modules/App/App.js b/client/modules/App/App.js
index 937c7713..9aa7c1df 100644
--- a/client/modules/App/App.js
+++ b/client/modules/App/App.js
@@ -1,4 +1,4 @@
-import React from 'react';
+import React, { PropTypes } from 'react';
 import { connect } from 'react-redux';
 import DevTools from './components/DevTools';
 
@@ -22,4 +22,8 @@ class App extends React.Component {
   }
 }
 
+App.propTypes = {
+  children: PropTypes.object
+};
+
 export default connect()(App);
diff --git a/client/modules/User/components/LoginForm.js b/client/modules/User/components/LoginForm.js
index a9196935..c7ae778b 100644
--- a/client/modules/User/components/LoginForm.js
+++ b/client/modules/User/components/LoginForm.js
@@ -31,7 +31,7 @@ function LoginForm(props) {
 
 LoginForm.propTypes = {
   fields: PropTypes.shape({
-    username: PropTypes.string.isRequired,
+    email: PropTypes.string.isRequired,
     password: PropTypes.string.isRequired
   }).isRequired,
   handleSubmit: PropTypes.func.isRequired,