Page MenuHomePhabricator
Differential D20710

Fix missing display cell in daemon summary table
ClosedPublic
Actions

Authored by epriestley on Aug 12 2019, 6:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jan 6, 4:41 PM
Unknown Object (File)
Mon, Dec 30, 11:46 AM
Unknown Object (File)
Mon, Dec 30, 11:46 AM
Unknown Object (File)
Mon, Dec 30, 9:48 AM
Unknown Object (File)
Thu, Dec 19, 3:50 PM
Unknown Object (File)
Dec 12 2024, 4:00 AM
Unknown Object (File)
Dec 9 2024, 3:57 PM
Unknown Object (File)
Dec 7 2024, 5:50 PM
View All 81 Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13374: Daemons summary table is missing a cell in the "Temporary Failures" row
Commits
rPc092492a53e0: Fix missing display cell in daemon summary table
Summary

Fixes T13374. The "Temporary Failures" row is missing a cell definiton from the addition of "Average Queue Time".

Test Plan

Viewed "/daemon/" with some temporary failures and and odd number of rows above the "Temporary Failures" row. Saw cell properly zebra-striped.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Aug 12 2019, 6:12 PM
Harbormaster completed remote builds in B23247: Diff 49387.Aug 12 2019, 6:14 PM
epriestley requested review of this revision.Aug 12 2019, 6:14 PM
This revision was not accepted when it landed; it landed in state Needs Review.Aug 12 2019, 7:46 PM
Closed by commit rPc092492a53e0: Fix missing display cell in daemon summary table (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rPc092492a53e0: Fix missing display cell in daemon summary table.

Revision Contents
Changeset List

PathSize
src/
applications/
daemon/
controller/
1 line

Diff 49390

View Options

src/applications/daemon/controller/PhabricatorDaemonConsoleController.php

Loading...
Phabricator · Built by Phacility