Page MenuHomePhabricator

[drydock/core] Show owner link on Drydock leases
AbandonedPublic

Authored by epriestley on Jul 23 2015, 1:37 AM.
Tags
None
Referenced Files
F15506481: D13687.diff
Tue, Apr 15, 11:37 AM
F15488068: D13687.diff
Thu, Apr 10, 7:04 PM
F15480474: D13687.id33087.diff
Tue, Apr 8, 2:18 PM
F15464849: D13687.id33087.diff
Wed, Apr 2, 12:55 PM
F15464358: D13687.id.diff
Wed, Apr 2, 8:49 AM
F15464185: D13687.id33788.diff
Wed, Apr 2, 7:07 AM
F15462912: D13687.diff
Tue, Apr 1, 6:53 PM
F15460990: D13687.id33087.diff
Tue, Apr 1, 4:08 AM
Subscribers

Details

Reviewers
hach-que
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T2015: Implement Drydock
Summary

Ref T2015. Drydock leases have an optional owner PHID which is set by Harbormaster, but it isn't actually show on the interface anywhere. This field is extremely useful for debugging, as it allows users to immediately find out what builds are holding what resources open.

Show the owner handle (if present) on all lease lists and on the lease controller.

Test Plan

Viewed pages, saw the owner handle.

Event Timeline

hach-que retitled this revision from to [drydock/core] Show owner link on Drydock leases.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
epriestley edited edge metadata.

I think this can be simplified to just $viewer->renderHandleLink($owner_phid) plus an empty state now?

This revision now requires changes to proceed.Sep 23 2015, 6:18 PM
epriestley abandoned this revision.
epriestley edited reviewers, added: hach-que; removed: epriestley.

See simpler D14191.