Differential D12845

Strip GMail French quote header
ClosedPublic
Actions

Authored by epriestley on May 14 2015, 7:08 PM.

Details

Reviewers

btrahan

Maniphest Tasks

T8199: Inbound email kind-of working (but kind of only).

Commits

Restricted Diffusion Commit
rPb34d5bf44ba9: Strip GMail French quote header

Summary

Ref T8199. Add pattern matching for GMail French.

Test Plan

Switched Gmail to french.
Replied to some mail.
Extracted quoted text header.
Added and executed unit test.

Diff Detail

Repository

rP Phabricator

Branch

frenchquote

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 5986
Build 6006: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 30899.May 14 2015, 7:08 PM

epriestley retitled this revision from to Strip GMail French quote header.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

epriestley added a task: T8199: Inbound email kind-of working (but kind of only)..

Herald added a subscriber: epriestley. · View Herald TranscriptMay 14 2015, 7:08 PM

Look perfect to me !

btrahan accepted this revision.May 14 2015, 7:58 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.May 14 2015, 7:58 PM

Closed by commit rPb34d5bf44ba9: Strip GMail French quote header (authored by epriestley, committed by epriestley). · Explain WhyMay 14 2015, 8:04 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

metamta/

parser/

PhabricatorMetaMTAEmailBodyParser.php

6 lines

__tests__/

PhabricatorMetaMTAEmailBodyParserTestCase.php

7 lines

Diff 30899

View Options

src/applications/metamta/parser/PhabricatorMetaMTAEmailBodyParser.php

View Options

src/applications/metamta/parser/tests/PhabricatorMetaMTAEmailBodyParserTestCase.php

Strip GMail French quote headerClosedPublicActions