Page MenuHomePhabricator

Add "parent" and "ancestor" information to the project.search API
ClosedPublic

Authored by epriestley on Jan 8 2017, 7:48 PM.
Tags
None
Referenced Files
F17699380: D17154.diff
Tue, Jul 15, 7:57 PM
F17620956: D17154.diff
Wed, Jul 9, 9:20 PM
Unknown Object (File)
Sun, Jul 6, 3:56 AM
Unknown Object (File)
Thu, Jul 3, 8:06 AM
Unknown Object (File)
Sat, Jun 28, 12:13 PM
Unknown Object (File)
Thu, Jun 26, 5:12 AM
Unknown Object (File)
Sun, Jun 22, 2:13 AM
Unknown Object (File)
Thu, Jun 19, 10:32 PM
Subscribers
None

Details

Summary

Ref T12074.

  • Adds a new "parent" property on main results. This shows an abbreviated version of the project's parent, or null if the project is a root project.
  • Adds a new "ancestor" attachment to pull the entire ancestor list.
  • Adds a new "depth" property on main results.
  • You can use "parent" or "depth" to tell if a project is a subproject or not.

These attempt to balance convenience, power, and performance: the full ancestor list can be big so I made it an attachment, but the other stuff isn't too big and is cheap and seems reasonable to always include.

Test Plan

In API results:

  • Saw null parent (root projects) and non-null parent (subprojects/milestones).
  • Used "ancestors" attchment, got full list of ancestors.
  • Saw appropriate "depth" values.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Jan 8 2017, 8:13 PM
This revision was automatically updated to reflect the committed changes.