User Auth

[ronilbhatia]

User Auth

Functionality

• [x] The main button with logo should link to /
• [ ] Has working demo login
• [x] Smooth, bug free navigation

Before Login

• [x] The /login page should not display a link to Log In. Same for /signup
• [ ] Going to a random route /#/oweiniouewbrviuwebv should redirect or display a 404 page
• [ ] Errors should display for both /signup and /login.
• [ ] Errors should clear when moving between /signup and /login.
• [x] Can sign up a user
• [x] Can sign in as a user
• [x] Can log out a user
• [x] Can't sign up with the same username/email
• [x] Pressing enter after filling out the session form should use the form data, not the demo user

After Login

• [x] Should not be able to visit /login or /signup
• [x] Should be able to refresh the page and still be logged in

Style

• [x] The site should look exactly like the actual site
• [x] All relevant elements should have cursor: pointer on hover
• [ ] When errors are displayed, most elements should not move around the page (particularly input boxes)
• [ ] Remove Redux logger and all console.log()'s from production

Seeds

• [ ] Adequate and appropriate seeds

[ronilbhatia]

Hey Alec, please see my comments below:

• VERY IMPORTANT - You need demo login functionality. A recruiter visiting your site should easily be able to click a demo button and use your site rather than being forced to sign up. Make this demo button available on both the signup and login pages and probably also the splash page. The easier it is to login as a demo user the better.
• You are not rendering errors to the screen. If the user attempts to signup with an email address already in use, or their password is too short, errors need to be rendered to the screen letting them know as such. Similarly with login, if the user tries logging in with invalid credentials an error needs to be rendered. Once you get this working, make sure that these errors do not persist when you switch between the signup and login forms (dispatching a clearErrors action in componentWillUnmount will do the trick).
• If I visit a random route I should be redirected back to the home page, but currently that is not happening.
• Remove your Redux logger from production. You can accomplish this by refactoring your configureStore method to look like this

  const configureStore = (preloadedState = {}) => {
  let middleware = [thunk];
  if (process.env.NODE_ENV !== 'production') {
  middleware = [...middleware, logger];
  }
  return createStore(
  rootReducer,
  preloadedState,
  applyMiddleware(...middleware)
  );
  };