PA Review: User Authentication

[nramabad]

New Account Creation, Login & Demo Account

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

[nramabad]

Before I forget, why is the heroku site more responsive on Chrome vs Safari? (specifically logout functionality)

[nramabad]

Also, is there a better way of seeding my demo account login credentials?

[evhumphrey]

What's the link to your heroku app? Could you update https://github.com/nramabad/Clone-ify/wiki

[evhumphrey]

RE: Chrome vs Safari -> Site seems to look the same on my computer across browsers

Feedback:

• Add hover styling to your links (when logged in)
• On user page, add hover styling to the buttons
• Make sure to clear your errors between forms. EX: try to login with invalid credentials -> you see errors -> go to sign up form -> the errors should not persist (they currently do)

[evhumphrey]

Feedback:

Auth Forms

• (styling) I would make the error text a tiny bit bigger
• (styling) I would have a little bit of a margin on the top of the errors box
• (BUG) I can't sign up on the heroku site, there's an error in the console

browse/featured frontend route

• (styling) when logged out, center the auth buttons in the side bar
• (styling bug) when hovering on sign up button, both buttons increase in size

General

• You're home page should look more like: https://www.spotify.com/us/. I think you just need an image, and need to make the page a bit bigger
• You shouldn't be able to go to https://clone-ify.herokuapp.com/#/aljdflkjadsflkajsflsjk (a jibberish route). You can have a switch where the last thing in it is just a path=/