diff options
| author | Pierre de La Morinerie <pierre@capitainetrain.com> | 2014-02-18 13:34:57 +0400 |
|---|---|---|
| committer | Pierre de La Morinerie <pierre@capitainetrain.com> | 2014-02-19 21:25:18 +0400 |
| commit | d8a24003cbd66dc80393dae46299c5734d50b5e0 (patch) | |
| tree | 33c1f4135237967604b2111de3ee82054d42b935 /app/mailers | |
| parent | 44ab156ef45fb1a566395b4d98afe796b28d21c9 (diff) | |
Don't write the project namespace in email subjects
For instance, the email "Subject" field changes from:
Mattt / Ground Control | Saving issue doesn't work sometimes (#1234)
to:
Ground Control | Saving issue doesn't work sometimes (#1234)
Rationale:
* Most people are receiving email notifications about a single fork:
the project activity, issues and merge requests are happening either
in a root repository or in a fork, but less often in several forks.
* It removes noise from the email subject. The namespace of the project
can still be read in the email body.
* For Entreprise users that have a single namespace "MyEntreprise",
having this repeated in every email subject is very noisy.
Diffstat (limited to 'app/mailers')
| -rw-r--r-- | app/mailers/notify.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/mailers/notify.rb b/app/mailers/notify.rb index 666cef4d3cb..3a4c9cf73b9 100644 --- a/app/mailers/notify.rb +++ b/app/mailers/notify.rb @@ -56,7 +56,7 @@ class Notify < ActionMailer::Base # => "Lorem ipsum | Dolor sit amet" def subject(*extra) subject = "" - subject << "#{@project.name_with_namespace} | " if @project + subject << "#{@project.name} | " if @project subject << extra.join(' | ') if extra.present? subject end |