PA Review: User Authentication

[natkozak]

• [x] Backend: DB, model, controller, views
• [x] Redux Loop: ajax, actions, reducer
• [x] Presentational Components and Containers
• [x] Styling
• [x] Smooth, bug-free navigation
• [x] Adequate and appropriate seeds

[samwalker191]

Thanks for submitting this issue! Feel free to use this checklist to make sure you got everything we want for the auth feature. Styling included. Once your app is live, I can review the MVP

User Auth

Functionality

• [x] The main button with logo should link to /
• [x] 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
• [x] Going to a random route /#/oweiniouewbrviuwebv should redirect or display a 404 page
• [x] Errors should display for both /signup and /login.
• [x] 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

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

  Seeds

• [x] Adequate and appropriate seeds

[samwalker191]

Looks good!

A few notes below:

1. I think the splash page is good for now! But, you will have to come back at some point to update it so it looks a little more like slacks. It's just empty for now. No changes needed for now.

2. Make sure you are consistent with your fonts, I think the buttons on the session form is using a different font family

3. The error messages could have some more space between them and the inputs. They are a bit too squished together.