Page MenuHomePhabricator
Differential D10112

Use standard infrastructure for Audit email generation
ClosedPublic
Actions

Authored by epriestley on Aug 1 2014, 7:38 PM.
Tags
None
Referenced Files
F13960054: D10112.id24318.diff
Mon, Oct 14, 9:48 PM
Unknown Object (File)
Tue, Oct 1, 11:43 AM
Unknown Object (File)
Tue, Sep 24, 9:40 PM
Unknown Object (File)
Tue, Sep 24, 4:12 PM
Unknown Object (File)
Sun, Sep 22, 5:09 PM
Unknown Object (File)
Sun, Sep 22, 5:06 PM
Unknown Object (File)
Sun, Sep 22, 5:04 PM
Unknown Object (File)
Sun, Sep 22, 5:03 PM
View All Files
Subscribers
epriestley
staticshock

Details

Reviewers
joshuaspence
btrahan
Maniphest Tasks
T4896: Move Audit to ApplicationTransactions
Commits
Restricted Diffusion Commit
rP64736264a62d: Use standard infrastructure for Audit email generation
Summary

Ref T4896. Replace custom stuff with standard stuff.

Test Plan
  • Sent a bunch of email and it all looked sensible/correct.
  • Made sure to test inlines, specifically, as they're a bit tricky.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Diff 24318.Aug 1 2014, 7:38 PM
epriestley retitled this revision from to Use standard infrastructure for Audit email generation.
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 1 2014, 7:38 PM
Harbormaster completed remote builds in B1988: Diff 24318.Aug 1 2014, 7:40 PM
btrahan accepted this revision.Aug 1 2014, 8:09 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Aug 1 2014, 8:09 PM
epriestley closed this revision.Aug 2 2014, 7:06 AM
epriestley updated this revision to Diff 24335.

Closed by commit rP64736264a62d (authored by @epriestley).

staticshock added a subscriber: staticshock.Sep 8 2014, 7:55 PM
staticshock added inline comments.
src/applications/audit/storage/PhabricatorAuditTransaction.php
57

Clsoed

missedthat

Revision Contents
Changeset List

PathSize
src/
applications/
audit/
editor/
225 lines
119 lines
storage/
51 lines

Diff 24335

View Options

src/applications/audit/editor/PhabricatorAuditCommentEditor.php

Loading...
View Options

src/applications/audit/editor/PhabricatorAuditEditor.php

Loading...
View Options

src/applications/audit/storage/PhabricatorAuditTransaction.php

Loading...
Phabricator · Built by Phacility