mailgun support messageids. supportsMessageIDHeader accidentally returned false.
ClosedPublic
Actions

Authored by talshiri on Jun 24 2014, 11:18 PM.

Details

Reviewers

Group Reviewers

Commits

Restricted Diffusion Commit
rP6f82725a9931: mailgun support messageids. supportsMessageIDHeader accidentally returned false.

Summary

The adapter was mostly copy-paste, and I missed the supportsMessageIDHeader stuff.

Test Plan

Sent a message, checked headers.

Diff Detail

Repository

rP Phabricator

Branch

mailgun_supports_message_id

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 1371
Build 1371: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

talshiri updated this revision to Diff 23328.Jun 24 2014, 11:18 PM

talshiri retitled this revision from to mailgun support messageids. supportsMessageIDHeader accidentally returned false..

talshiri updated this object.

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

talshiri added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 24 2014, 11:18 PM

Herald added subscribers: Korvin, epriestley. · View Herald Transcript

Harbormaster completed remote builds in B1371: Diff 23328.Jun 24 2014, 11:19 PM

epriestley accepted this revision.Jun 24 2014, 11:21 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jun 24 2014, 11:21 PM

Closed by commit rP6f82725a9931 (authored by @talshiri, committed by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

metamta/

adapter/

PhabricatorMailImplementationMailgunAdapter.php

2 lines

Diff 23328

View Options

src/applications/metamta/adapter/PhabricatorMailImplementationMailgunAdapter.php

mailgun support messageids. supportsMessageIDHeader accidentally returned false.ClosedPublicActions