Page MenuHomePhabricator

Add harbormaster.buildplan.search api method
ClosedPublic

Authored by timhirsh on Nov 2 2018, 1:50 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 15 2024, 2:38 AM
Unknown Object (File)
Feb 12 2024, 12:00 AM
Unknown Object (File)
Feb 3 2024, 7:44 PM
Unknown Object (File)
Jan 9 2024, 9:22 AM
Unknown Object (File)
Jan 9 2024, 9:22 AM
Unknown Object (File)
Jan 6 2024, 1:14 PM
Unknown Object (File)
Dec 27 2023, 10:40 PM
Unknown Object (File)
Dec 24 2023, 10:22 PM
Subscribers

Details

Summary

This revision adds a conduit search method for build plans. Other api methods (eg: harbormaster.build.search) support build plan phid's as a constraint, but they weren't exposed anywhere, so this provides a way to fetch them.

Test Plan

Used the api console to run some searches. Output:

{
  "data": [
    {
      "id": 1,
      "type": "HMCP",
      "phid": "PHID-HMCP-q2c25wvegzdkxs7gzor6",
      "fields": {
        "name": "my build plan",
        "planStatus": "active",
        "dateCreated": 1538085249,
        "dateModified": 1538085249,
        "policy": {
          "view": "users",
          "edit": "admin"
        }
      },
    {
      "id": 1,
      "type": "HMCP",
      "phid": "PHID-HMCP-q2c25wvegzdkxs7gzor6",
      "fields": {
        "name": "my build plan",
        "status": {
          "value": "active"
        },
        "dateCreated": 1538085249,
        "dateModified": 1538085249,
        "policy": {
          "view": "users",
          "edit": "admin"
        }
      },
      "attachments": {}
    },
    ...
  ],
  "maps": {},
  "query": {
    "queryKey": null
  },
  "cursor": {
    "limit": 100,
    "after": null,
    "before": null,
    "order": null
  }
}

Diff Detail

Repository
rP Phabricator
Branch
build_plan_search_api
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21073
Build 28635: Run Core Tests
Build 28634: arc lint + arc unit

Event Timeline

Couple of minor inlines, happy to upstream this otherwise.

src/applications/harbormaster/conduit/HarbormasterBuildPlanSearchAPIMethod.php
16

For consistency, capitalize "Harbormaster".

src/applications/harbormaster/storage/configuration/HarbormasterBuildPlan.php
231

Let's just call this status and make it a dictionary with the status constant in "value" so we can add name, icon, color, etc., later if necessary. See DifferentialRevision for a similar object.

...
"status": {
  "value": <whatever getPlanStatus() returns>
}
...
This revision now requires changes to proceed.Nov 2 2018, 1:57 AM

capitalization & make status a dictionary

epriestley added inline comments.
src/applications/harbormaster/storage/configuration/HarbormasterBuildPlan.php
223

Oh, make this, uh, map<string ,wild> now too. (It's just documentation for human consumers.)

This revision is now accepted and ready to land.Nov 2 2018, 2:29 AM

oops, update field spec too

thanks Evan!

This revision was automatically updated to reflect the committed changes.