Apply Diffusion reply email directly with transaction editor
ClosedPublic
Actions

Authored by epriestley on Aug 2 2014, 7:19 PM.

Details

Reviewers

joshuaspence
btrahan

Maniphest Tasks

T4896: Move Audit to ApplicationTransactions

Commits

Restricted Diffusion Commit
rP25acf5d1304c: Apply Diffusion reply email directly with transaction editor

Summary

Ref T4896. Invoke the new editor directly instead of in a roundabout way when handling Audit email.

Test Plan

Used bin/mail receive-test to simulate mail, saw comment post with proper content source.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 24354.Aug 2 2014, 7:19 PM

epriestley retitled this revision from to Apply Diffusion reply email directly with transaction editor.

epriestley updated this object.

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

epriestley added reviewers: btrahan, joshuaspence.

epriestley added a task: T4896: Move Audit to ApplicationTransactions.

Herald added a subscriber: epriestley. · View Herald TranscriptAug 2 2014, 7:19 PM

Harbormaster completed remote builds in B2009: Diff 24354.Aug 2 2014, 7:20 PM

btrahan accepted this revision.Aug 2 2014, 9:07 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Aug 2 2014, 9:07 PM

Closed by commit rP25acf5d1304c (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

audit/

mail/

PhabricatorAuditReplyHandler.php

32 lines

Diff 24364

View Options

src/applications/audit/mail/PhabricatorAuditReplyHandler.php

Apply Diffusion reply email directly with transaction editorClosedPublicActions