euglazer/lovetothink — pull request #102 by ootoovak (2014-12-15 20:34:25+00:00)
Title: Code review

# TESTS!!!!!!!! NEED TESTS!!!!!!!!!

Start fixing this right away!!! You know how to write them. You know they are important. You know know first hand the benefits they give. Don't let this or code quality slide for imagined speed or features.

# "God" Classes

This is where one class becomes huge with functionality and is connected to everything. This often happens on projects especially on the User class. Techniques you can reference to start pulling things apart:
[Therapeutic Refactoring](https://www.youtube.com/watch?v=J4dlF0kcThQ)
[7 Patterns to Refactor Fat ActiveRecord Models](http://blog.codeclimate.com/blog/2012/10/17/7-ways-to-decompose-fat-activerecord-models/)

# Rest of the notes in comments below
@euglazer @sarahannkelleher