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
F15421862: D20710.id49390.diff
Sat, Mar 22, 2:48 AM
F15386955: D20710.id49390.diff
Sat, Mar 15, 1:34 AM
Unknown Object (File)
Feb 16 2025, 11:48 PM
Unknown Object (File)
Feb 9 2025, 7:58 AM
Unknown Object (File)
Feb 9 2025, 7:58 AM
Unknown Object (File)
Feb 9 2025, 7:58 AM
Unknown Object (File)
Feb 9 2025, 7:58 AM
Unknown Object (File)
Feb 5 2025, 1:42 PM
View All 99 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