ylqjgm
/
canvas-lms
mirror of https://github.com/instructure/canvas-lms.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

I18n support for 'Update your notification' in email 

refs: GH-1333, CNVS-43588

Test Plan:
1. Select an user with email address set. Change the language preference
   of this user to a language which is not English( Simplified Chinese,
   aka '简体中文' for example)
2. Trigger a action to send an email to this user.
3. In the bottom of the notification email, 'Update your notification
   settings' shows in English. Although it has already been properly
   translated in locale files.
   (messages.layout.email.html.click_to_preference)

Change-Id: I9613bbf32a3214a8158e7969182ae640144485b6
Reviewed-on: https://gerrit.instructure.com/163115
Reviewed-by: Steven Burnett <sburnett@instructure.com>
QA-Review: Steven Burnett <sburnett@instructure.com>
Product-Review: Steven Burnett <sburnett@instructure.com>
Tested-by: Jenkins
Reviewed-by: Jacob Fugal <jacob@instructure.com>
This commit is contained in:
gaojun 2018-08-21 11:16:39 +08:00 committed by Steven Burnett
parent 3bd849d4f3
commit 2f848edaa0
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/messages/_layout.email.html.erb
View File

@ -95,7 +95,7 @@ Margin is capitalized to fix Outlook.com
<%= content(:footer_link).html_safe %> &nbsp;|&nbsp;
<% end %>
<a href="<%= communication_profile_url %>" style="white-space: nowrap;"><%= t 'click_to_preferences', "Update your notification settings" %></a>
<a href="<%= communication_profile_url %>" style="white-space: nowrap;"><%= t "Update your notification settings" %></a>
</td>
</tr>