PA Review: { User Authentication }

[chinweenie]

• [ ] Backend: DB, model, controller, views
• [ ] Redux Loop: ajax, actions, reducer
• [ ] Presentational Components (session form)
• [ ] Styling (modal)
• [ ] Smooth, bug-free navigation
• [ ] Deployed on Heroku: https://myepsy.herokuapp.com

[andywynkoop]

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
• [ ] 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

• [x] 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)
• [ ] Remove Redux logger and all console.log()'s from production

Seeds

• [x] Adequate and appropriate seeds

[andywynkoop]

Hey Winnie, here are a few suggestions for your project so far (based on what's up on heroku)

• [x] At least on my screen, the modal you have to display user information is unreachable - it goes away before I can hover over it
• [ ] The site doesn't need to be fully responsive, but when I view it on a more narrow screen, the elements shouldn't stack on top of each other, it's better for them to just go off-screen
• [x] You probably already know this, but some of the styling is off for the store manager page - errors need to be styled, some buttons are off-center, etc