Page MenuHomePhabricator

Introduce "bridged" objects
ClosedPublic

Authored by epriestley on Mar 21 2016, 6:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 9:57 AM
Unknown Object (File)
Mon, Apr 8, 7:23 AM
Unknown Object (File)
Wed, Apr 3, 8:26 AM
Unknown Object (File)
Mon, Apr 1, 3:16 PM
Unknown Object (File)
Mar 13 2024, 2:10 AM
Unknown Object (File)
Mar 12 2024, 6:16 PM
Unknown Object (File)
Mar 12 2024, 5:25 PM
Unknown Object (File)
Mar 12 2024, 3:11 PM
Subscribers

Details

Reviewers
chad
Maniphest Tasks
T10537: Nuance Infrastructure
Commits
Restricted Diffusion Commit
rP47dedfb152d0: Introduce "bridged" objects
Summary

Ref T10537. These are objects which are bound to some external object, like a Maniphest task which is a representation of a GitHub issue.

This doesn't do much yet and may change, but my thinking is:

  • I'm putting these on-object instead of on edges because I think we want to actively change the UI for them (e.g., clearly call out that the object is bridged) but don't want every page to need to do extra queries in the common case where zero bridged objects exist anywhere in the system.
  • I'm making these one-to-one, more or less: an issue can't be bridged to a bunch of tasks, nor can a bunch of tasks be bridged to a single issue. Pretty sure this makes sense? I can't come up with any reasonable, realistic cases where you want a single GitHub issue to publish to multiple different tasks in Maniphest.
  • Technically, one type of each bridgable object could be bridged, but I expect this to never actually occur. Hopefully.
Test Plan

Ran storage upgrade, loaded some pages.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Introduce "bridged" objects.
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.Mar 21 2016, 8:05 PM
This revision was automatically updated to reflect the committed changes.