Page MenuHomePhabricator
Differential D13701

Modularize "Send an Email" Herald actions
ClosedPublic
Actions

Authored by epriestley on Jul 24 2015, 3:37 PM.
Tags
None
Referenced Files
F14052950: D13701.diff
Fri, Nov 15, 11:45 AM
F14045172: D13701.id33113.diff
Wed, Nov 13, 1:56 AM
F14038369: D13701.diff
Sun, Nov 10, 11:13 PM
F14022714: D13701.diff
Wed, Nov 6, 7:00 PM
F14001487: D13701.id33279.diff
Fri, Oct 25, 8:32 AM
F13964654: D13701.id.diff
Oct 15 2024, 9:13 PM
F13964653: D13701.diff
Oct 15 2024, 9:13 PM
F13964216: D13701.id33279.diff
Oct 15 2024, 7:56 PM
View All Files
Subscribers
epriestley
joshuaspence

Details

Reviewers
btrahan
Maniphest Tasks
T8726: Modularize Herald fields and actions
Commits
Restricted Diffusion Commit
rP51fead17cf13: Modularize "Send an Email" Herald actions
Summary

Ref T8726. No surprises.

Test Plan

Created rules using both action variants, applied upgrade, saw rules still work correctly.

Screen Shot 2015-07-24 at 8.32.28 AM.png (152×544 px, 22 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 33113.Jul 24 2015, 3:37 PM
epriestley retitled this revision from to Modularize "Send an Email" Herald actions.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T8726: Modularize Herald fields and actions.
Herald added a subscriber: epriestley. · View Herald TranscriptJul 24 2015, 3:37 PM
btrahan accepted this revision.Jul 24 2015, 5:42 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jul 24 2015, 5:42 PM
btrahan added inline comments.Jul 24 2015, 5:42 PM
src/applications/subscriptions/herald/PhabricatorSubscriptionsHeraldAction.php
15–22

nice

joshuaspence added a subscriber: joshuaspence.Jul 25 2015, 11:26 PM
Closed by commit rP51fead17cf13: Modularize "Send an Email" Herald actions (authored by epriestley, committed by epriestley). · Explain WhyAug 3 2015, 9:52 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 33279

View Options

resources/sql/autopatches/20150724.herald.2.sql

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/differential/herald/HeraldDifferentialRevisionAdapter.php

Loading...
View Options

src/applications/diffusion/herald/HeraldCommitAdapter.php

Loading...
View Options

src/applications/diffusion/herald/HeraldPreCommitAdapter.php

Loading...
View Options

src/applications/herald/adapter/HeraldAdapter.php

Loading...
View Options

src/applications/maniphest/herald/HeraldManiphestTaskAdapter.php

Loading...
View Options

src/applications/metamta/herald/PhabricatorMetaMTAEmailHeraldAction.php

Loading...
View Options

src/applications/metamta/herald/PhabricatorMetaMTAEmailOthersHeraldAction.php

Loading...
View Options

src/applications/metamta/herald/PhabricatorMetaMTAEmailSelfHeraldAction.php

Loading...
View Options

src/applications/phriction/herald/PhrictionDocumentHeraldAdapter.php

Loading...
View Options

src/applications/subscriptions/herald/PhabricatorSubscriptionsAddSelfHeraldAction.php

Loading...
View Options

src/applications/subscriptions/herald/PhabricatorSubscriptionsAddSubscribersHeraldAction.php

Loading...
View Options

src/applications/subscriptions/herald/PhabricatorSubscriptionsHeraldAction.php

Loading...
View Options

src/applications/subscriptions/herald/PhabricatorSubscriptionsRemoveSelfHeraldAction.php

Loading...
View Options

src/applications/subscriptions/herald/PhabricatorSubscriptionsRemoveSubscribersHeraldAction.php

Loading...
View Options

src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php

Loading...
Phabricator · Built by Phacility