Closed jideokus closed 7 years ago
@jideokus Amazing work, thanks so much. So far it looks good to me, gonna double check in the morning and fix the conflicts, then merge. Thanks!
Sure let me know if I should go ahead to resolve the conflicts
Hi, thanks yes, @jideokus if you can solve the conflicts we can make the review faster and approve. thanks!
Sorry just seeing this. Will resolve them now.
Conflicts resolved @dvidsilva. Good to go
@jideokus looks like there was a typo on pacakge.json, must have been uncaught on the merge coz it had no conflict. And a couple linting things, fixing that and merging. Thanks :D
Please fill out this template, ignore or delete the parts that are not relevant to your code.
Solves issue Number: #1
Description
Created user model and added in login and sign up buttons Added middleware to check authentication and user role Added user resource controller to handle sign up and login
Todos
The following things are missing for feature completeness:
Deploy Notes
We added the following packages or dependencies:
Steps to Test or Reproduce
Impacted Areas in Application
The following features or routes might be affected