New email notification chains not working

Discussion of open issues, suggestions and bugs regarding code review tool for Visual Studio – Review Assistant
Post Reply
jessejacob
Posts: 4
Joined: Mon 01 Jun 2015 14:05

New email notification chains not working

Post by jessejacob » Mon 08 Apr 2019 18:08

Greetings! I see the release notes for RA 4 include the text "In addition, now all the notifications on one review are combined into message chains." I just upgraded to 4.1.223 and I noticed that RA emails now include the SMTP header "references" with a unique value for each review; however, there is no "in-reply-to" header with the same value, so e.g. Outlook 2016 doesn't display the messages as a conversation. In effect the emails look nicer than older RA versions, but they are still completely unchained and unrelated to on-another and just jumble up the inbox like before. By way of comparison, github enterprise includes both the references and in-reply-to headers on every message--even the initial review, interestingly. When you tested this feature, what email client let you get away with only using the references header and not including in-reply-to?

RFC2822 (section 3.6.4, https://www.ietf.org/rfc/rfc2822.txt ) includes details on how to implement these fields correctly--long story short this feature isn't going to work as planned without both headers. Is this a regression in RA 4.1.223? I didn't use any of the initial 4.x releases except this one and came right from 3.6 latest. We're extremely interested in seeing this feature in action because RA emails are just all over the place and difficult to follow, so we'd really appreciate some help understanding what's going on. Ideally this feature should allow any email client (that understands how to use those fields for threading) to chain all messages so you have one thread per review.

Thanks,

Jesse

mstislavs
Devart Team
Posts: 102
Joined: Mon 16 Jan 2017 12:48

Re: New email notification chains not working

Post by mstislavs » Wed 10 Apr 2019 11:34

Thank you for reporting the problem. We are investigating it.

Best regards,
Mstislav Sakidon
Devart Team
www.devart.com

mstislavs
Devart Team
Posts: 102
Joined: Mon 16 Jan 2017 12:48

Re: New email notification chains not working

Post by mstislavs » Mon 15 Apr 2019 10:25

Combining Review Assistant notifications into message chains in Outlook will be fixed in the next version. (issue R128201)

Best regards,
Mstislav Sakidon
Devart Team
www.devart.com

jessejacob
Posts: 4
Joined: Mon 01 Jun 2015 14:05

Re: New email notification chains not working

Post by jessejacob » Mon 15 Apr 2019 18:25

Great! Any idea when that will be released/what the number will be, or how I can track the status of that issue?

Jesse

mstislavs
Devart Team
Posts: 102
Joined: Mon 16 Jan 2017 12:48

Re: New email notification chains not working

Post by mstislavs » Mon 22 Apr 2019 12:12

jessejacob wrote: Mon 15 Apr 2019 18:25 Great! Any idea when that will be released/what the number will be, or how I can track the status of that issue?
Yes of course, issue #R128201. You can track it in the revisions history https://www.devart.com/review-assistant ... story.html. This week we plan to publish release where we fixed integration to VS2019 and after it we will start new version. This bug with notifications will be fixed in the new version.

Best regards,
Mstislav Sakidon
Devart Team
www.devart.com

mstislavs
Devart Team
Posts: 102
Joined: Mon 16 Jan 2017 12:48

Re: New email notification chains not working

Post by mstislavs » Tue 06 Aug 2019 14:08

Hi,

We've investigated the issue in Outlook 2016 again and we don't see any problems when the "Show as Conversations" option is enabled in Outlook 2016 https://take.ms/Q2w9g

Please attach the screenshots demonstrating the problem or contact us using the following form
https://www.devart.com/company/contactform.html

Best regards,
Mstislav Sakidon
Devart Team
www.devart.com

Post Reply