Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

# User Auth #7

Open
15 of 17 tasks
Evan-Leon opened this issue Nov 2, 2021 · 2 comments
Open
15 of 17 tasks

# User Auth #7

Evan-Leon opened this issue Nov 2, 2021 · 2 comments

Comments

@Evan-Leon
Copy link
Collaborator

Evan-Leon commented Nov 2, 2021 •
edited
Loading

Functionality

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

Before Login

  • 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.
  • Can sign up a user
  • Can sign in as a user
  • Can log out a user
  • Can't sign up with the same username/email
  • Pressing enter after filling out the session form should use the form data, not the demo user

After Login

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

Style

  • The elements are positioned correctly on the page
  • Remove Redux logger and all console.log()'s from production

Seeds

  • Adequate and appropriate
@elliot-wilson
Copy link
Owner

16.5/17 done! My anti-"random route" mechanism is a little janky.

@Evan-Leon
Copy link
Collaborator Author

Great job so far!

  • I can't seem to get login errors to work
  • errors don't clear going from signup to login

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Evan-Leon @elliot-wilson