-
Notifications
You must be signed in to change notification settings - Fork 17
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.
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
Class 23 - Week 2+3 Project #3
Comments
Here it is my project: https://github.com/PolyannaMeira/node-template |
Here is a link to my forked repo: https://github.com/GKing3/node-template |
Here is a link to my forked repo: https://github.com/stefan-000/node-template |
My repo link: https://github.com/Kev-Ose/node-template |
My repo click this link |
Repo link: https://github.com/Hammad1820/node-template |
Link to my forked repo: https://github.com/juryp/node-template |
@PolyannaMeira the recipes routes look good, are you planning to implement the user routes ? |
@GKing3 good start, but you're missing some recipe controllers that are not implemented such as |
@stefan-000 do you need any help ? You need to update the SQL queries to perform the correct operation, handle the errors in case the correct parameters are not passed to the route, and return the correct HTTP status code in the response. |
@Kev-Ose I don't see any changes compared to the template, do you need any help ? |
@nibishakajean this looks great, congrats ! |
@Smita81 be careful that you created a duplicate |
@Hammad1820 I don't see any changes on the main branch, did you push your code ? |
@pallavisarwar I left some comments on your pull request HackYourFutureBelgium/node-template#10 |
@Gemechis10 I don't see any changes to the template, did you push your code ? |
@juryp This looks good, but what is the |
@maeligg thanks for your review. here is the link after corrections. |
Please leave below a comment with a link to your week 2 project.
Instructions
The text was updated successfully, but these errors were encountered: