Page MenuHomePhabricator
Differential D16863

Don't send Phurl mail to the URL object itself
ClosedPublic
Actions

Authored by epriestley on Nov 15 2016, 5:01 PM.
Tags
None
Referenced Files
F14052398: D16863.diff
Fri, Nov 15, 8:48 AM
F14038442: D16863.diff
Sun, Nov 10, 11:41 PM
F14032029: D16863.id40606.diff
Sat, Nov 9, 12:37 PM
F14032028: D16863.id40603.diff
Sat, Nov 9, 12:37 PM
F14032027: D16863.id.diff
Sat, Nov 9, 12:36 PM
F14024219: D16863.diff
Thu, Nov 7, 7:15 AM
F14019770: D16863.diff
Tue, Nov 5, 11:37 PM
F14009453: D16863.diff
Wed, Oct 30, 5:49 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T11868: Phurl URLs send email to themselves (???)
Commits
rP508d86aab6cf: Don't send Phurl mail to the URL object itself
Summary

Fixes T11868. This is silly and does not make sense.

Test Plan

Edited a Phurl URL, verified mail only went to me, not to the object itself.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 40603.Nov 15 2016, 5:01 PM
epriestley retitled this revision from to Don't send Phurl mail to the URL object itself.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T11868: Phurl URLs send email to themselves (???).
chad accepted this revision.Nov 15 2016, 5:05 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 15 2016, 5:05 PM
Closed by commit rP508d86aab6cf: Don't send Phurl mail to the URL object itself (authored by epriestley, committed by epriestley). · Explain WhyNov 15 2016, 5:12 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
phurl/
editor/
4 lines

Diff 40606

View Options

src/applications/phurl/editor/PhabricatorPhurlURLEditor.php

Loading...
Phabricator · Built by Phacility