CadmiumRed40 / FindAFlick

Group Movie App
4 stars 3 forks source link

Updated user model #25

Open AllieGie opened 2 years ago

AllieGie commented 2 years ago

Updates Made:

Things to note:

pan-alex commented 2 years ago

All changes up until the last commit look good. The most recent commit introduces a breaking change in signup.ejs because the data was split into two different forms. This causes an error anytime someone tries to sign up as data like the email, username, and password aren't included in the request.

Since the last commit is mostly for styling I would suggest reverting that commit so that we can merge this PR. Then add the styling changes in a new branch. You should be able to revert that commit by running: git reset --hard d1bb0d1