Page MenuHomePhabricator
Differential D19031

Add an "Acting user" field to Herald
ClosedPublic
Actions

Authored by epriestley on Feb 8 2018, 5:41 PM.
Tags
None
Referenced Files
F15529999: D19031.id45634.diff
Wed, Apr 23, 12:28 AM
F15483817: D19031.id.diff
Wed, Apr 9, 2:38 PM
F15433609: D19031.diff
Mar 25 2025, 12:01 AM
F15421165: D19031.id45632.diff
Mar 21 2025, 9:24 PM
F15394716: D19031.id.diff
Mar 16 2025, 12:23 AM
F15394134: D19031.id45634.diff
Mar 15 2025, 11:20 PM
F15394130: D19031.id45632.diff
Mar 15 2025, 11:20 PM
F15394127: D19031.id.diff
Mar 15 2025, 11:20 PM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T7804: Implement an "Acting User" Herald condition
T13053: Plans: Mail Tags and Failover
Commits
rPbca9c08953bd: Add an "Acting user" field to Herald
Summary

Ref T13053. Fixes T7804. Adds "Acting user" so you can have "always email me" stuff skip things you did or keep an eye on suspicious interns.

For the test console, the current user is the acting user.

For pushes, the pusher is the acting user.

Test Plan

Wrote acting user rules, triggered them via test console and via multiple actors on real objects.

Diff Detail

Repository
rP Phabricator
Branch
mail7
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19427
Build 26274: Run Core Tests
Build 26273: arc lint + arc unit

Event Timeline

epriestley created this revision.Feb 8 2018, 5:41 PM
Harbormaster completed remote builds in B19427: Diff 45632.Feb 8 2018, 5:42 PM
epriestley requested review of this revision.Feb 8 2018, 5:42 PM
epriestley added a child revision: D19032: Render object remarkup references in a text context as "Dxxx <uri>".Feb 8 2018, 5:50 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 8 2018, 5:52 PM
Closed by commit rPbca9c08953bd: Add an "Acting user" field to Herald (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
2 lines
applications/
diffusion/
engine/
6 lines
herald/
adapter/
10 lines
controller/
1 line
field/
32 lines
transactions/
editor/
1 line

Diff 45632

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/diffusion/engine/DiffusionCommitHookEngine.php

Loading...
View Options

src/applications/herald/adapter/HeraldAdapter.php

Loading...
View Options

src/applications/herald/controller/HeraldTestConsoleController.php

Loading...
View Options

src/applications/herald/field/HeraldActingUserField.php

Loading...
View Options

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

Loading...
Phabricator · Built by Phacility