Differential D13836

Update Dashboards for handleRequest
ClosedPublic
Actions

Authored by chad on Aug 10 2015, 3:17 AM.

Details

Reviewers

epriestley

Maniphest Tasks

T8628: Modernize AphrontControllers to implement handleRequest

Commits

Restricted Diffusion Commit
rPb92e6d3582f6: Update Dashboards for handleRequest

Summary

Updates Dashboards app

Test Plan

Click on Manage Dashboard, page load a.ok

Diff Detail

Repository

rP Phabricator

Branch

process-dashboard

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 7551
Build 8142: [Placeholder Plan] Wait for 30 Seconds
Build 8141: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 33416.Aug 10 2015, 3:17 AM

chad retitled this revision from to Update Dashboards for handleRequest.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added a reviewer: epriestley.

Herald added a subscriber: Korvin. · View Herald TranscriptAug 10 2015, 3:17 AM

epriestley accepted this revision.Aug 10 2015, 9:36 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Aug 10 2015, 9:36 AM

Closed by commit rPb92e6d3582f6: Update Dashboards for handleRequest (authored by chad, committed by chad). · Explain WhyAug 10 2015, 4:07 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in D14223: Correct a Dashboard status constant in a migration.Oct 2 2015, 1:13 PM

epriestley mentioned this in rP3c4b05bcd4b6: Correct a Dashboard status constant in a migration.Oct 2 2015, 4:17 PM

joshuaspence added a task: T8628: Modernize AphrontControllers to implement handleRequest.Oct 18 2015, 6:58 AM

Revision Contents
Changeset List

Path

Size

src/

applications/

dashboard/

controller/

PhabricatorDashboardManageController.php

14 lines

Diff 33416

View Options

src/applications/dashboard/controller/PhabricatorDashboardManageController.php

Update Dashboards for handleRequestClosedPublicActions