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

RESOLVED FIXED in Thunderbird 47.0

Status

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: mkmelin, Assigned: mkmelin)

Tracking

Trunk
Thunderbird 47.0
Bug Flags:
in-testsuite +

Thunderbird Tracking Flags

(thunderbird45 fixed, thunderbird46 fixed, thunderbird47 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Created attachment 8711819 [details] [diff] [review]
bugXXX_auto-reply-refs.patch

"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+
(Assignee)

Comment 2

3 years ago
https://hg.mozilla.org/comm-central/rev/63ee59a8e786 -> FIXED
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-thunderbird47: --- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 47.0
(Assignee)

Comment 3

3 years ago
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?

Updated

3 years ago
Attachment #8711819 - Flags: approval-comm-aurora? → approval-comm-aurora+
(Assignee)

Updated

3 years ago
status-firefox47: affected → ---

Comment 4

3 years ago
Aurora (TB 46):
https://hg.mozilla.org/releases/comm-aurora/rev/8057891037ba
status-thunderbird45: --- → affected
status-thunderbird46: --- → fixed
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+

Updated

3 years ago
status-thunderbird45: affected → fixed

Updated

3 years ago
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.