User Email preferences being overriden

nat · June 28, 2022, 7:29am

Ah, so this happened to me yesterday and I can confirm that it’s deliberate…

The supporting tests and comments indicates so

Code:

discourse/discourse/blob/aa38dcd26313e65229597099dddc115f37d3ee20/lib/email/receiver.rb#L1398-L1403


      
          def enable_email_pm_setting(user)
            # ensure user PM emails are enabled (since user is posting via email)
            if !user.staged && user.user_option.email_messages_level == UserOption.email_level_types[:never]
              user.user_option.update!(email_messages_level: UserOption.email_level_types[:always])
            end
          end

Tests:

github.com

discourse/discourse/blob/aa38dcd26313e65229597099dddc115f37d3ee20/spec/lib/email/receiver_spec.rb#L1029-L1035


      
          it "reenables user's PM email notifications when user emails new topic to group" do
            user = Fabricate(:user, email: "existing@bar.com")
            user.user_option.update_columns(email_messages_level: UserOption.email_level_types[:never])
            expect { process(:group_existing_user) }.to change(Topic, :count)
            user.reload
            expect(user.user_option.email_messages_level).to eq(UserOption.email_level_types[:always])
          end

Essentially when we send an email to a group, despite having set “Never” to that option, it will always set it to “Always”. The idea is that if we send an email, we’d want to receive the email response in our email inbox as well.

The sort-of answer here is to just use Discourse to send PMs to groups.

Topic		Replies	Views
Notification emails not being sent for Chat as per user preference bug chat	27	1155	July 4, 2023
Changing Username Stops Email Notifications support	26	2243	September 30, 2017
Email Notification recipients unclear when PM is sent to multiple users feature email	39	10658	May 7, 2018
"Daily updates" option for mailing list mode feature	42	7223	June 28, 2017
Please restore email digests feature	47	6493	June 20, 2017

User Email preferences being overriden

Related Topics