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
Unknown Object (File)
Jun 18 2025, 10:05 PM
Unknown Object (File)
May 23 2025, 5:04 AM
Unknown Object (File)
May 7 2025, 9:18 PM
Unknown Object (File)
Apr 29 2025, 5:03 AM
Unknown Object (File)
Apr 26 2025, 1:28 PM
Unknown Object (File)
Apr 23 2025, 3:21 AM
Unknown Object (File)
Apr 21 2025, 7:19 AM
Unknown Object (File)
Apr 19 2025, 11:36 PM
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.