Page MenuHomePhabricator

Give "PhabricatorUserEmail" a PHID
ClosedPublic

Authored by epriestley on Thu, Nov 14, 7:31 PM.

Details

Summary

Ref T13444. To interact meaningfully with "DestructionEngine", objects need a PHID. The "UserEmail" object currently does not have one (or a real "Query").

Provide basic PHID support so "DestructionEngine" can interact with the object more powerfully.

Test Plan
  • Ran migrations, checked data in database, saw sensible PHIDs assigned.
  • Added a new email address to my account, saw it get a PHID.

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 created this revision.Thu, Nov 14, 7:31 PM
Owners added a subscriber: Restricted Owners Package.Thu, Nov 14, 7:31 PM
epriestley requested review of this revision.Thu, Nov 14, 7:32 PM
This revision was not accepted when it landed; it landed in state Needs Review.Tue, Nov 19, 5:41 PM
This revision was automatically updated to reflect the committed changes.