radar/forem — pull request #197 by jeremyong (2012-04-14 19:49:36+00:00)
Title: User subscription preferences

This branch adds a conditional to the after_save :subscribe_user on post.rb.

The conditional proc checks the forem_auto_subscribe column on the users table. I've also added a migration for this column which defaults to false.

I'm ok with the default being changed to something else but I personally think it's irritating for a user to have to keep mashing the unsubscribe button every time they make a new reply if they don't want to subscribe in the first place. With this, it's easy to allow the user to customize their subscription preferences later.

I've modified the tests and factories slightly to reflect this change. Let me know if everything looks good on your end.

Cheers,
Jeremy