Page MenuHomePhabricator

Modularize "Send an Email" Herald actions
ClosedPublic

Authored by epriestley on Jul 24 2015, 3:37 PM.

Details

Summary

Ref T8726. No surprises.

Test Plan

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

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; 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.
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

This revision was automatically updated to reflect the committed changes.