Closed Bug 1242676 Opened 5 years ago Closed 5 years ago

"Reply with template" filter should add References: and In-Reply-To: headers to the reply mail.

Categories

(MailNews Core :: Filters, defect)

defect
Not set
normal

Tracking

(thunderbird45 fixed, thunderbird46 fixed, thunderbird47 fixed)

RESOLVED FIXED
Thunderbird 47.0
Tracking Status
thunderbird45 --- fixed
thunderbird46 --- fixed
thunderbird47 --- fixed

People

(Reporter: mkmelin, Assigned: mkmelin)

Details

Attachments

(1 file)

"Reply with template" filter should add References: and In-Reply-To: headers to the reply mail so that you can see what mail it was a reply to, and threading will work.
Attachment #8711819 - Flags: review?(rkent)
Comment on attachment 8711819 [details] [diff] [review]
bugXXX_auto-reply-refs.patch

Review of attachment 8711819 [details] [diff] [review]:
-----------------------------------------------------------------

OK looks good with one nit.

::: mailnews/compose/src/nsMsgCompose.cpp
@@ +1800,5 @@
>  
>    nsCOMPtr<nsIPrefBranch> prefs (do_GetService(NS_PREFSERVICE_CONTRACTID, &rv));
>    NS_ENSURE_SUCCESS(rv, rv);
>  
>    // If we are forwarding inline, mime did already setup the compose fields therefore we should stop now

Can you correct this comment since you have expanded the meaning of this section?
Attachment #8711819 - Flags: review?(rkent) → review+
https://hg.mozilla.org/comm-central/rev/63ee59a8e786 -> FIXED
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 47.0
Comment on attachment 8711819 [details] [diff] [review]
bugXXX_auto-reply-refs.patch

[Approval Request Comment]
User impact if declined: auto-replies not threaded
Testing completed (on c-c, etc.): landed on trunk with test
Risk to taking this patch (and alternatives if risky): not risky, should land after trunk baking
Attachment #8711819 - Flags: approval-comm-beta?
Attachment #8711819 - Flags: approval-comm-aurora?
Attachment #8711819 - Flags: approval-comm-aurora? → approval-comm-aurora+
Comment on attachment 8711819 [details] [diff] [review]
bugXXX_auto-reply-refs.patch

http://hg.mozilla.org/releases/comm-esr45/rev/024ed64816aa
Attachment #8711819 - Flags: approval-comm-beta? → approval-comm-beta+
Attachment #8711819 - Flags: approval-comm-beta+ → approval-comm-esr45+
You need to log in before you can comment on or make changes to this bug.