From f2e0059c96ad40faf59865b7916b7259b02a6479 Mon Sep 17 00:00:00 2001 From: rgarcia Date: Mon, 18 Apr 2016 19:58:01 +0200 Subject: [PATCH 1/2] activates newsletter by default --- db/migrate/20160418172919_activate_newsletter_by_default.rb | 5 +++++ db/schema.rb | 4 ++-- 2 files changed, 7 insertions(+), 2 deletions(-) create mode 100644 db/migrate/20160418172919_activate_newsletter_by_default.rb diff --git a/db/migrate/20160418172919_activate_newsletter_by_default.rb b/db/migrate/20160418172919_activate_newsletter_by_default.rb new file mode 100644 index 000000000..cdee85f1b --- /dev/null +++ b/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 diff --git a/db/schema.rb b/db/schema.rb index 35d28fc78..223a14d47 100644 --- a/db/schema.rb +++ b/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" From 08db8a6b1741cf2145b778e81eec063393ef7d4e Mon Sep 17 00:00:00 2001 From: rgarcia Date: Mon, 18 Apr 2016 20:08:05 +0200 Subject: [PATCH 2/2] fixes specs --- spec/models/user_spec.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index c63959a64..4d1b269ec 100644 --- a/spec/models/user_spec.rb +++ b/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 - expect(subject.newsletter).to eq(false) + it 'should be true by default' do + expect(subject.newsletter).to eq(true) end end end