From 96dfd7186dc6ee5976e80a1884a467f30b96b753 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Mar 28 2015 18:21:45 +0000 Subject: When notifying about a new comment set the correct headers The correct headers been, the Message-Id containing the uid of the ticket/pull-request and the id of the comment itself and the In-Reply-To header allowing threading --- diff --git a/pagure/lib/notify.py b/pagure/lib/notify.py index c46874f..41dfbcb 100644 --- a/pagure/lib/notify.py +++ b/pagure/lib/notify.py @@ -156,6 +156,7 @@ New comment: text, 'Update to issue #%s `%s`' % (comment.issue.id, comment.issue.title), ','.join(mail_to), + mail_id=comment.mail_id, in_reply_to=comment.issue.mail_id, ) @@ -386,11 +387,11 @@ New comment: 'Update to pull-Request #%s `%s`' % ( comment.pull_request.id, comment.pull_request.title), ','.join(mail_to), - mail_id=comment.pull_request.mail_id, + mail_id=comment.mail_id, + in_reply_to=comment.pull_request.mail_id, ) - def notify_new_email(email, user): ''' Ask the user to confirm to the email belong to them. '''