Merge pull request #139 from fac-14/authentication

routes now restricted, but not with jwt > #123
fac-14 · Nov 29, 2018 · 85cd6ea · 85cd6ea
2 parents faa1564 + ea39212
commit 85cd6ea
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/client/src/App.js b/client/src/App.js
@@ -72,7 +72,15 @@ class App extends Component {
           <Route
             exact={true}
             path="/profile/:id/add"
-            render={props => <Add {...props} />}
+            render={props => {
+              return isAuthenticated ? (
+                <Add {...props} />
+              ) : (
+                <Redirect
+                  to={{ pathname: "/login", state: { from: props.location } }}
+                />
+              );
+            }}
           />
           <Route exact={true} path="/find" component={Discovery} />
           <Route exact={true} path="/login" component={LogIn} />

diff --git a/client/src/components/LogIn.js b/client/src/components/LogIn.js
@@ -11,7 +11,7 @@ export default class LogIn extends React.Component {
     allowFetchSubmit: false
   };
 
-  /* 
+  /*
   Possible functions:
     sanitiseInputs() // to remove any "illegal" characters from email and pw inputs
     handleBackendValidation // likely not necessary and implemented as part of the fetch
@@ -112,6 +112,8 @@ export default class LogIn extends React.Component {
             type="email"
             onBlur={this.checkEmailValid}
             required
+            autocomplete="on"
+            x-autocompletetype="email"
           />
           <label htmlFor="password">
             <h3>Password</h3>