Fix a typo in the new "ApplicationEmailHeraldField"
ClosedPublic
Actions

Authored by epriestley on Jul 7 2015, 1:13 PM.

Details

Reviewers

btrahan
chad

Commits

Restricted Diffusion Commit
rPb36d4025cbf8: Fix a typo in the new "ApplicationEmailHeraldField"

Summary

This rule isn't quite right.

Test Plan

Shipped email in with a rule in effect, saw correct value in transcript.

Diff Detail

Repository

rP Phabricator

Branch

emailrule1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 7179
Build 7403: [Placeholder Plan] Wait for 30 Seconds
Build 7402: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 32816.Jul 7 2015, 1:13 PM

epriestley retitled this revision from to Fix a typo in the new "ApplicationEmailHeraldField".

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added reviewers: btrahan, chad.

Herald added a subscriber: epriestley. · View Herald TranscriptJul 7 2015, 1:13 PM

chad accepted this revision.Jul 7 2015, 1:18 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Jul 7 2015, 1:18 PM

Closed by commit rPb36d4025cbf8: Fix a typo in the new "ApplicationEmailHeraldField" (authored by epriestley, committed by epriestley). · Explain WhyJul 7 2015, 1:22 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

metamta/

herald/

PhabricatorMetaMTAApplicationEmailHeraldField.php

2 lines

Diff 32816

View Options

src/applications/metamta/herald/PhabricatorMetaMTAApplicationEmailHeraldField.php

Fix a typo in the new "ApplicationEmailHeraldField"ClosedPublicActions