- Issue created by @Aziza Anwari
- last update
about 1 year ago 10 pass, 2 fail - last update
about 1 year ago 10 pass, 2 fail - Status changed to RTBC
12 months ago 12:52pm 10 April 2024 - last update
11 months ago 10 pass, 2 fail - 🇺🇸United States k_a_l
This patch wasn't applying cleanly for me so I updated it for latest 3.6.
- 🇺🇸United States k_a_l
My mistake, was not applying due to another patch being applied.
The last submitted patch, 4: unable_to_send_email_with_reply_to_not_set_error-3410890-4.patch, failed testing. View results →
- Status changed to Needs review
3 months ago 7:44pm 8 January 2025 - 🇺🇸United States kevinquillen
For Acquia Platform Email, reply-to header is required or you will get this error (Amazon SES service). The patch made this work, although I do not understand why you have to uncheck 'Disable site email notification' to get reply-to to be set. Can't it just be set by default to the site mail that is sending the message out and not include the site mail as a (main) recipient?
$recipient = ''; $reply_to = NULL; if (!$notification->disableSiteMail()) { $recipient = \Drupal::config('system.site')->get('mail'); $reply_to = $recipient; }
Why not simply
$reply_to = \Drupal::config('system.site')->get('mail');
?