Skip to content

Commit 3b24d37

Browse files
Merge pull request #5572 from mausamp/fix-5567/distribution-mailer-wording-change
fix-5567 : Update the text in the distribution update mailer ( wording change )
2 parents 69a2487 + 9adebed commit 3b24d37

2 files changed

Lines changed: 6 additions & 3 deletions

File tree

app/views/distribution_mailer/_distribution_changes.html.erb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
<p>Here are some Items that we need change based on Human Essentials</p>
1+
<p>We've had to change some items on your distribution</p>
22
<ul style="list-style-type:none">
33
<% if @distribution_changes[:updates].any? %>
44
<li>Items Updated</li>

spec/mailers/distribution_mailer_spec.rb

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -96,8 +96,11 @@
9696
it "renders the body with changes that happened in the distribution" do
9797
distribution = create(:distribution, organization: user.organization, comment: "Distribution comment", partner: partner, delivery_method: :delivery)
9898
mail = DistributionMailer.partner_mailer(organization, distribution, 'test subject', distribution_changes)
99-
expect(mail.body.encoded).to match(distribution_changes[:removed][0][:name])
100-
expect(mail.body.encoded).to match(distribution_changes[:updates][0][:name])
99+
html = html_body(mail)
100+
101+
expect(html).to match(distribution_changes[:removed][0][:name])
102+
expect(html).to match(distribution_changes[:updates][0][:name])
103+
expect(html).to match("We've had to change some items on your distribution")
101104
end
102105
end
103106

0 commit comments

Comments
 (0)