Merge pull request #1073 from consul/newsletter

activates newsletter by default

db/migrate/20160418172919_activate_newsletter_by_default.rb

@@ -0,0 +1,5 @@
+class ActivateNewsletterByDefault < ActiveRecord::Migration
+  def change
+    change_column :users, :newsletter, :boolean, default: true
+  end
+end

db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20160415150524) do
+ActiveRecord::Schema.define(version: 20160418172919) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -416,7 +416,7 @@ ActiveRecord::Schema.define(version: 20160415150524) do
     t.string   "erase_reason"
     t.datetime "erased_at"
     t.boolean  "public_activity",                      default: true
-    t.boolean  "newsletter",                           default: false
+    t.boolean  "newsletter",                           default: true
     t.integer  "notifications_count",                  default: 0
     t.boolean  "registering_with_oauth",               default: false
     t.string   "locale"

spec/models/user_spec.rb

@@ -79,8 +79,8 @@ describe User do
     end
 
     describe 'subscription_to_website_newsletter' do
-      it 'should be false by default' do
+      it 'should be true by default' do
-        expect(subject.newsletter).to eq(false)
+        expect(subject.newsletter).to eq(true)
       end
     end
   end