ridler / myCal

A REST API to manage calendar events.
0 stars 0 forks source link

David Aragon's evaluation #4

Open DavidQL opened 9 years ago

DavidQL commented 9 years ago

Nice job, the app works great. One problem I had during setup is that there was no DB directory specified, so I had to manually create one and then start mongo with that path: mongod --dbpath data. I had this issue on another project too so I don't think it's a problem with your app, but it may be nice to mention in the README.

One suggestion I would make is to break up app.js into several smaller files. That app is responsible for a lot of infrastructure and business logic, and it would be easier to digest if the discreet components were in their own files.