Skip to content

Commit 2368f3e

Browse files
authored
Merge pull request #2868 from Alex-Jordan/reply-all
prompt email recipient to use reply-all when there were multiple reci…
2 parents f8a7d9c + e9a534a commit 2368f3e

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

templates/ContentGenerator/Feedback/feedback_email.html.ep

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,15 @@
1111
% } elsif ($set) {
1212
Message from <%= $user->full_name %> (<%= $user->user_id %>) via WeBWorK at
1313
<%= $ce->{institutionName} %>
14-
(sent from <%= link_to format_set_name_display($set->set_id) => $emailableURL %>)
14+
(sent from <%= link_to format_set_name_display($set->set_id) => $emailableURL %>).
1515
% } else {
1616
Message from <%= $user->full_name %> (<%= $user->user_id %>) via WeBWorK at
1717
<%= $ce->{institutionName} %> (sent from <%= link_to 'this page' => $emailableURL %>).
1818
% }
1919
</p>
20+
% if ($numRecipients > 1) {
21+
<p><%= maketext('Message delivered to multiple recipients. Consider using reply-all.') =%></p>
22+
% }
2023
% if ($feedback) {
2124
<div style="border: 1px solid lightgray; padding: 1rem; margin-bottom: 1rem; border-radius: 0.375rem">
2225
% for (split /\n\r?/, $feedback) {

templates/ContentGenerator/Feedback/feedback_email.txt.ep

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,10 @@ Feedback sent from <%= format_set_name_display($set->set_id) %>:
1111
Feedback sent from:
1212
% }
1313
<%== $emailableURL %>
14+
% if ($numRecipients > 1) {
15+
16+
<%= maketext('Message delivered to multiple recipients. Consider using reply-all.') %>
17+
% }
1418

1519
% if ($feedback) {
1620
<%== $user->full_name %> (<%== $user->user_id %>) wrote:

0 commit comments

Comments
 (0)