From 23979a05aa95d3b43216e2f3f13caf67678c20e0 Mon Sep 17 00:00:00 2001 From: Chad Little Date: Mon, 4 Apr 2016 19:26:05 +0000 Subject: [PATCH] Upate notifications for newPage Summary: Single callsite, swap to `newPage` Test Plan: Visit page, see same status message. Also remove device ready flag. Reviewers: epriestley Reviewed By: epriestley Subscribers: Korvin Differential Revision: https://secure.phabricator.com/D15610 --- .../PhabricatorNotificationStatusController.php | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/src/applications/notification/controller/PhabricatorNotificationStatusController.php b/src/applications/notification/controller/PhabricatorNotificationStatusController.php index eb79a3a8c5..49d3fb9d9b 100644 --- a/src/applications/notification/controller/PhabricatorNotificationStatusController.php +++ b/src/applications/notification/controller/PhabricatorNotificationStatusController.php @@ -24,15 +24,12 @@ final class PhabricatorNotificationStatusController $crumbs = $this->buildApplicationCrumbs(); $crumbs->addTextCrumb(pht('Status')); - return $this->buildApplicationPage( - array( - $crumbs, - $status, - ), - array( - 'title' => pht('Notification Server Status'), - 'device' => false, - )); + $title = pht('Notification Server Status'); + + return $this->newPage() + ->setTitle($title) + ->setCrumbs($crumbs) + ->appendChild($status); } private function renderServerStatus(array $status) {