Differential D13686

[drydock/core] Only show active resources and leases in controllers
AbandonedPublic
Actions

Authored by epriestley on Jul 23 2015, 1:17 AM.

Tags

None

Referenced Files

	F14706798: D13686.id33787.diff
	Fri, Jan 17, 3:03 AM

	F14706796: D13686.id33086.diff
	Fri, Jan 17, 3:03 AM

	F14706795: D13686.id.diff
	Fri, Jan 17, 3:03 AM

	F14706779: D13686.diff
	Fri, Jan 17, 2:59 AM

	Unknown Object (File)
	Fri, Jan 3, 8:05 AM

	Unknown Object (File)
	Fri, Dec 27, 2:49 AM

	Unknown Object (File)
	Dec 17 2024, 8:06 AM

	Unknown Object (File)
	Dec 16 2024, 5:48 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T2015: Implement Drydock

Summary

Ref T2015. These controllers get quite impossible to view when there are thousands of closed leases or resources. Instead, only show the active leases and resources so it's easier for users to evaluate whether or not it's safe to manually close a resource.

Test Plan

Viewed it.

Diff Detail

Branch

master

Lint

Lint Passed

Unit

Test Failures

Build Status

Buildable 7360
Build 7760: [Placeholder Plan] Wait for 30 Seconds
Build 7759: arc lint + arc unit

Unit TestsFailed
View All

	Time	Test
	467 ms	PhabricatorCelerityTestCase::Unknown Unit Message ("")
	918 ms	PhabricatorLibraryTestCase::Unknown Unit Message ("")
	8 ms	PhabricatorConduitTestCase::Unknown Unit Message ("")
	0 ms	PhabricatorInfrastructureTestCase::Unknown Unit Message ("")
	1 ms	PhabricatorInfrastructureTestCase::Unknown Unit Message ("")
		View Full Test Results (2 Failed · 5 Passed)

Event Timeline

hach-que updated this revision to Diff 33086.Jul 23 2015, 1:17 AM

hach-que retitled this revision from to [drydock/core] Only show active resources and leases in controllers.

hach-que updated this object.

hach-que edited the test plan for this revision. (Show Details)

hach-que added a reviewer: epriestley.

hach-que added a task: T2015: Implement Drydock.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJul 23 2015, 1:17 AM

Herald added subscribers: Korvin, epriestley. · View Herald Transcript

Rebase

Obsoleted by D14157 + D14158.

epriestley abandoned this revision.Sep 24 2015, 8:54 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

drydock/

controller/

DrydockBlueprintViewController.php

6 lines

DrydockResourceViewController.php

5 lines

Diff 33086

src/applications/drydock/controller/DrydockBlueprintViewController.php

Loading...

src/applications/drydock/controller/DrydockResourceViewController.php

Loading...