Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Sometimes, you could want the reply-to address by different from the sender's one...
Thanks
Comment | File | Size | Author |
---|---|---|---|
#16 | mimemail-1469022-16-d7.patch | 5.32 KB | sgabe |
#16 | mimemail-1469022-16-d6.patch | 6 KB | sgabe |
#11 | mimemail-1469022-11-d7.patch | 5.64 KB | sgabe |
#11 | mimemail-1469022-11-d6.patch | 6.71 KB | sgabe |
#10 | mimemail-1469022-10.patch | 5.64 KB | sgabe |
Comments
Comment #1
PhilYAm I missing something so obvious?
Comment #2
sgabe CreditAttribution: sgabe commentedDo you mean to allow to set a Reply-To address for the Rules actions? If you are using Mime Mail within a custom module you can add the Reply-To header manually.
Comment #3
PhilYyes, in a Rules "Send HTML e-mail" action I find To, CC and CCI fields but no Reply-To
Thanks
Comment #4
sgabe CreditAttribution: sgabe commentedChanging issue settings.
Comment #5
Wouter Van den Bosch CreditAttribution: Wouter Van den Bosch commentedHaving this feature would be very helpful for us as well.
Comment #7
das-peter CreditAttribution: das-peter commentedAttached patch adds a reply-to parameter to the mimemail rules action.
Further it generalizes the address header handling.
Use-case for the reply-to for us is that our customer gets notifications about commerce orders.
The notification uses the shop-customers mail address as sender, that way the booking-agents can reply directly to the notification mail.
"Unfortunately" our customer has a restrictive mail-server that checks if a sender mail address is valid.
That means if the shop-customer inadvertently enters an invalid mail-address the booking-agents won't get the notification and thus don't have a chance to act as quick as normal.
Adding the reply-to header could solve this issue by still allowing the agents to reply directly to the notification mail.
Comment #8
sgabe CreditAttribution: sgabe commented@das-peter: Patch looks good, thank you! Would you do the same for the system actions as well and attach a new patch including that too?
Comment #9
das-peter CreditAttribution: das-peter commented@sgabe: Thank you for the feedback. Here's the extended patch.
I did also some cleanup using drupalcs - the second attached patch (
mimemail-coding-standards-cleanup.patch
) contains these changes and could be applied after the reply-to patch is applied.Comment #10
sgabe CreditAttribution: sgabe commentedThis line will throw an error because of the variable prefix in the index.
Otherwise it looks good, but we should also have a back port to D6.
Comment #11
sgabe CreditAttribution: sgabe commentedHere the header should be Bcc.
I am attaching new patches for D6 and D7.
Comment #12
sgabe CreditAttribution: sgabe commentedComment #13
das-peter CreditAttribution: das-peter commented@sgabe: Sorry for the sloppy copy/paste (
['$reply-to']
and thank you very much for your effort!Comment #14
sgabe CreditAttribution: sgabe commentedWould be appreciated if you could test it.
Comment #15
sgabe CreditAttribution: sgabe commentedPatches need to be rerolled.
Comment #16
sgabe CreditAttribution: sgabe commentedRerolled and committed to both branches.