Express conventions

[cameronjacoby]

@eerwitt - Can you look over the server file and Mongoose models to make sure they follow best-practices? I built this in the way we taught node/express/mongo for the last cohort and would love to use this API as a standard for how we're going to teach it this time. So I'd love to get your input to see if we should make any changes!

We have not separated controllers from routes in the past - everything just lives in the one server.js file. The module.exports can be confusing for students, so having just one server file is a way to make it less overwhelming. But if you feel strongly that it's an anti-pattern, we can definitely take a different approach.

[eerwitt]

Looking good so far, I understand the logic behind skipping module.exports and think this makes things much more clear.

I've been pushing people towards using http://usejsdoc.org/ formatted comments which I believe is a good habit especially for routes like this.

[cameronjacoby]

@eerwitt Thanks for taking a look! I'll try to refactor some of the comments as we keep working on and improving the API 😄