Resolve "Set “Reply-To” in diagram comment notification mails." #70

Merged
anselm merged 1 commit from 40-set-reply-to-in-diagram-comment-notification-mails into main 2023-06-24 01:45:58 +02:00
anselm commented 2023-06-24 01:34:22 +02:00 (Migrated from gitlab.anselms.net)

This is a fairly simple change. We must replace the call to django.core.mail.send_mail() with the instantiation of an EmailMessage object and call to its send() method, because send_mail() doesn't let us specify extra headers (EmailMessage has a separate reply_to attribute.)

Closes #40

This is a fairly simple change. We must replace the call to `django.core.mail.send_mail()` with the instantiation of an `EmailMessage` object and call to its `send()` method, because `send_mail()` doesn't let us specify extra headers (`EmailMessage` has a separate `reply_to` attribute.) Closes #40
anselm commented 2023-06-24 01:34:22 +02:00 (Migrated from gitlab.anselms.net)

assigned to @anselm

assigned to @anselm
anselm commented 2023-06-24 01:42:53 +02:00 (Migrated from gitlab.anselms.net)

added 1 commit

  • 388b6479 - feature: Include `Reply-To:` header in diagram/crib comment notification mail.

Compare with previous version

added 1 commit <ul><li>388b6479 - feature: Include `Reply-To:` header in diagram/crib comment notification mail.</li></ul> [Compare with previous version](/strathspey/ace4/-/merge_requests/14/diffs?diff_id=140&start_sha=02228efa4dc6a9dc68e0cf44ca64c4f2864f5779)
anselm commented 2023-06-24 01:44:36 +02:00 (Migrated from gitlab.anselms.net)

marked this merge request as ready

marked this merge request as **ready**
anselm commented 2023-06-24 01:45:58 +02:00 (Migrated from gitlab.anselms.net)

mentioned in commit b7b7b3d0c2

mentioned in commit b7b7b3d0c215466dbf3986711cd0c491e2fd0f2d
anselm (Migrated from gitlab.anselms.net) merged commit b7b7b3d0c2 into main 2023-06-24 01:45:58 +02:00
Sign in to join this conversation.
No description provided.