Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix email separators from mobile and web clients #109

Merged
merged 2 commits into from Mar 20, 2020

Conversation

EduardoRomo
Copy link
Collaborator

Different html schemas for outgoing emails produced parsing errors for emails sent from a mobile or web client. When the experimental html features were enabled, this produced that full email threads were added as a comment in ADO, instead of just the latest message.

Copy link
Contributor

@davisjms davisjms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix Eduardo! Have added some minor comments, please take a look and we should be good to go.

Mail2Bug/Email/EmailBodyProcessingUtils.cs Show resolved Hide resolved
Mail2Bug/Email/EmailBodyProcessingUtils.cs Show resolved Hide resolved
@EduardoRomo EduardoRomo merged commit 4c303d9 into microsoft:master Mar 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants