kaplali/parkstorm — pull request #53 by alycit (2014-07-09 15:18:54+00:00)
Title: [DO NOT MERGE] Code review complete

All..

Overall pretty good. There are comments in some of the files. See the 'changed files' for more info.

Key Points:
- Continue to maintain ideas of SRP/ Separation of concerns in JavaScript too.
- Extracting callbacks to named methods helps give a new reader of your code context.
- Remove commented out code
- Remove unnecessary puts
- Most of your helpers are empty. Don't need to keep them
- The naming of the files in db/fixtures are crazy. Make them more meaningful.
- Tests? Not many.