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)
Sun, Jun 1, 2:11 AM
Unknown Object (File)
Mon, May 26, 11:09 AM
Unknown Object (File)
Mon, May 26, 10:51 AM
Unknown Object (File)
Sun, May 25, 11:12 PM
Unknown Object (File)
May 8 2025, 1:42 AM
Unknown Object (File)
Apr 18 2025, 6:37 AM
Unknown Object (File)
Apr 17 2025, 6:57 AM
Unknown Object (File)
Apr 16 2025, 3:51 PM
View All 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