Page MenuHomePhabricator

Add Editor-based mail stamps: actor, via, silent, encrypted, new, mention, self-actor, self-mention
ClosedPublic

Authored by epriestley on Feb 5 2018, 7:03 PM.
Tags
None
Referenced Files
F13227387: D18995.diff
Mon, May 20, 3:02 AM
F13215709: D18995.diff
Fri, May 17, 7:18 PM
F13212483: D18995.diff
Fri, May 17, 6:46 AM
F13209268: D18995.id45560.diff
Thu, May 16, 11:00 PM
F13196047: D18995.diff
Sun, May 12, 10:53 PM
F13184813: D18995.id45560.diff
Sat, May 11, 12:23 AM
F13178622: D18995.diff
Wed, May 8, 8:32 PM
Unknown Object (File)
Sun, May 5, 6:28 AM
Subscribers
None

Details

Summary

Ref T13053. Adds more mail tags with information available on the Editor object.

Test Plan

Banged around in Maniphest, viewed the resulting mail, all the stamps seemed to align with reality.

Diff Detail

Repository
rP Phabricator
Branch
stamps3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19341
Build 26147: Run Core Tests
Build 26146: arc lint + arc unit

Event Timeline

amckinley added inline comments.
src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
204–206

Did this get picked up via a rebase?

This revision is now accepted and ready to land.Feb 5 2018, 10:58 PM
src/applications/transactions/engineextension/PhabricatorEditorMailEngineExtension.php
65

I added it so I could make the call here -- the flag was only internal with no getter before.

src/applications/transactions/engineextension/PhabricatorEditorMailEngineExtension.php
65

👍