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)
Thu, Apr 11, 8:06 AM
Unknown Object (File)
Sun, Mar 31, 6:18 AM
Unknown Object (File)
Sun, Mar 24, 11:13 PM
Unknown Object (File)
Jan 28 2024, 8:13 AM
Unknown Object (File)
Jan 28 2024, 3:05 AM
Unknown Object (File)
Jan 28 2024, 3:05 AM
Unknown Object (File)
Jan 28 2024, 3:05 AM
Unknown Object (File)
Jan 24 2024, 11:03 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.