Page MenuHomePhabricator

Fix for email parsing from Outlook clients.
ClosedPublic

Authored by wotte on May 6 2014, 10:26 PM.

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rPa57cbbea8c09: Fix for email parsing from Outlook clients.
Summary

D1239 got it mostly right, but some versions of Outlook apparently put a '> ' in front of the 'Original Message' marker, which the parser couln't grok.

Test Plan

Added a test case to the unit tests, applied the patch to my install and asked one of my heathen Outlook using colleagues to reply to a Conpherence post.

Diff Detail

Repository
rP Phabricator
Branch
outlook_fix
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 278
Build 278: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

wotte retitled this revision from to Fix for email parsing from Outlook clients..
wotte updated this object.
wotte edited the test plan for this revision. (Show Details)
wotte added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 7 2014, 12:53 AM
wrotte updated this revision to Diff 21372.
wrotte added a subscriber: wrotte.

Closed by commit rPa57cbbea8c09 (authored by @wrotte).