search

Kevin-Rau / breakfast

0 stars 4 forks source link

Edward Zhu's Evaluation *CRITICAL* #3

Open zhued opened 9 years ago

zhued commented 9 years ago

Issues:

breakfast.js

Line 3: skeleton.connect('mongodb://localhost/simple'); Skeleton is an Express framework-less app structure generator. It simply generates a skeleton of a basic structure. The ".connect" method does not exist in the skeleton package. You're not using skeleton properly. You need mongoose in order to call the connect method on a mongodb.

DoStuff.js

Line 1: var database= require('./mongo.js'); There is no js file that configs a MongoDB service. How did you get this assignment actually working with that still in the code?

Line 43: app.put('/person/:id', function(req, res){ Why did you decide on person/:id when your data in on breakfast foods and has nothing to do with people?

kenbod commented 9 years ago

Yes, it does appear that the assignment still needs some work to be functional. Would members of the team please respond to these comments and update the code so we have confidence your web service works?