Merge pull request #3527 from techAPJ/patch-3

FEATURE: include email_prefix in digest email subject instead of site title
This commit is contained in:
Régis Hanol 2015-06-03 19:27:08 +02:00
commit c9a341417c
2 changed files with 8 additions and 1 deletions

View File

@ -54,7 +54,7 @@ class UserNotifications < ActionMailer::Base
min_date = opts[:since] || @user.last_emailed_at || @user.last_seen_at || 1.month.ago
@site_name = SiteSetting.title
@site_name = SiteSetting.email_prefix.presence || SiteSetting.title
@header_color = ColorScheme.hex_for_name('header_background')
@last_seen_at = short_date(@user.last_seen_at || @user.created_at)

View File

@ -77,6 +77,13 @@ describe UserNotifications do
expect(subject.text_part.body.to_s).to be_present
end
it "includes email_prefix in email subject instead of site title" do
SiteSetting.email_prefix = "Try Discourse"
SiteSetting.title = "Discourse Meta"
expect(subject.subject).to match(/Try Discourse/)
expect(subject.subject).not_to match(/Discourse Meta/)
end
end
end