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
F15467989: D10112.id24335.diff
Thu, Apr 3, 7:02 PM
F15467988: D10112.id24318.diff
Thu, Apr 3, 7:02 PM
F15465376: D10112.id24318.diff
Wed, Apr 2, 6:02 PM
F15451904: D10112.id24335.diff
Sat, Mar 29, 12:30 AM
F15449039: D10112.id.diff
Fri, Mar 28, 8:19 AM
F15445209: D10112.diff
Thu, Mar 27, 12:52 PM
F15422614: D10112.diff
Mar 22 2025, 9:47 AM
F15383727: D10112.id24335.diff
Mar 14 2025, 5:41 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