Merge pull request #5305 from saymedia/master

Fix the sigil for merge request emails

2 files changed, 2 insertions, 2 deletions

diff --git a/app/mailers/emails/notes.rb b/app/mailers/emails/notes.rb
index 761b4c8161f..ba4f0dd862c 100644
--- a/app/mailers/emails/notes.rb
+++ b/app/mailers/emails/notes.rb
@@ -18,7 +18,7 @@ module Emails
       @note = Note.find(note_id)
       @merge_request = @note.noteable
       @project = @note.project
-      mail(to: recipient(recipient_id), subject: subject("note for merge request !#{@merge_request.iid}"))
+      mail(to: recipient(recipient_id), subject: subject("note for merge request ##{@merge_request.iid}"))
     end
 
     def note_wall_email(recipient_id, note_id)
diff --git a/spec/mailers/notify_spec.rb b/spec/mailers/notify_spec.rb
index 0787bdbea6f..666c6ccefff 100644
--- a/spec/mailers/notify_spec.rb
+++ b/spec/mailers/notify_spec.rb
@@ -320,7 +320,7 @@ describe Notify do
         it_behaves_like 'a note email'
 
         it 'has the correct subject' do
-          should have_subject /note for merge request !#{merge_request.iid}/
+          should have_subject /note for merge request ##{merge_request.iid}/
         end
 
         it 'contains a link to the merge request note' do