Page MenuHomePhabricator

Improve code structure of PHID fields in EditEngine
ClosedPublic

Authored by epriestley on Nov 29 2015, 9:45 PM.
Tags
None
Referenced Files
F14844625: D14601.diff
Sun, Feb 2, 5:13 PM
Unknown Object (File)
Thu, Jan 30, 12:48 PM
Unknown Object (File)
Wed, Jan 29, 9:11 PM
Unknown Object (File)
Wed, Jan 29, 9:11 PM
Unknown Object (File)
Wed, Jan 29, 9:11 PM
Unknown Object (File)
Fri, Jan 24, 11:38 PM
Unknown Object (File)
Fri, Jan 24, 10:09 AM
Unknown Object (File)
Thu, Jan 23, 5:32 PM
Subscribers
None

Details

Summary

Ref T9132. I had some hacks in place for dealing with Edge/Subscribers stuff. Clean that up so it's structured a little better.

Test Plan
  • Edited subscribers and projects.
  • Verified things still show up in Conduit.
  • Made concurrent edits (added a project in one window, removed it in another window, got a clean result with a correct merge of the two edits).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Improve code structure of PHID fields in EditEngine.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 30 2015, 3:00 PM
This revision was automatically updated to reflect the committed changes.