From 903090d1484408c8fe3fb91da28a6af17d3e288d Mon Sep 17 00:00:00 2001 From: Abe Hassan Date: Tue, 8 Oct 2013 11:32:26 -0700 Subject: [PATCH] Fix the sigil for merge request emails --- app/mailers/emails/notes.rb | 2 +- spec/mailers/notify_spec.rb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/mailers/emails/notes.rb b/app/mailers/emails/notes.rb index 761b4c816..ba4f0dd86 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 0787bdbea..666c6ccef 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 -- 2.11.0