diff --git a/app/controllers/email_controller.rb b/app/controllers/email_controller.rb index 1ae099f4131..b4aebf910cc 100644 --- a/app/controllers/email_controller.rb +++ b/app/controllers/email_controller.rb @@ -100,7 +100,8 @@ class EmailController < ApplicationController if params["unsubscribe_all"] user.user_option.update_columns(email_digests: false, email_level: UserOption.email_level_types[:never], - email_messages_level: UserOption.email_level_types[:never]) + email_messages_level: UserOption.email_level_types[:never], + mailing_list_mode: false) updated = true end diff --git a/spec/requests/email_controller_spec.rb b/spec/requests/email_controller_spec.rb index a0d0b4bb38d..bec3e9049dc 100644 --- a/spec/requests/email_controller_spec.rb +++ b/spec/requests/email_controller_spec.rb @@ -19,7 +19,8 @@ RSpec.describe EmailController do it 'can fully unsubscribe' do user.user_option.update_columns(email_digests: true, email_level: UserOption.email_level_types[:never], - email_messages_level: UserOption.email_level_types[:never]) + email_messages_level: UserOption.email_level_types[:never], + mailing_list_mode: true) post "/email/unsubscribe/#{key}.json", params: { unsubscribe_all: "1" } @@ -36,6 +37,7 @@ RSpec.describe EmailController do expect(user.user_option.email_digests).to eq(false) expect(user.user_option.email_level).to eq(UserOption.email_level_types[:never]) expect(user.user_option.email_messages_level).to eq(UserOption.email_level_types[:never]) + expect(user.user_option.mailing_list_mode).to eq(false) end it 'can disable mailing list' do