Differential D20408

Update "add panel" and "remove panel" Dashboard flows to the new panel storage format
ClosedPublic
Actions

Authored by epriestley on Apr 12 2019, 6:29 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Nov 8, 3:18 AM

	Unknown Object (File)
	Oct 16 2024, 4:07 AM

	Unknown Object (File)
	Oct 12 2024, 1:48 AM

	Unknown Object (File)
	Oct 9 2024, 12:05 PM

	Unknown Object (File)
	Oct 2 2024, 1:43 PM

	Unknown Object (File)
	Oct 1 2024, 5:37 PM

	Unknown Object (File)
	Oct 1 2024, 4:11 AM

	Unknown Object (File)
	Sep 30 2024, 6:09 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T13272: Remove the hard-coded homepage, and other Dashboards improvements

Commits

rP82c46f4b93bd: Update "add panel" and "remove panel" Dashboard flows to the new panel storage…

Summary

Depends on D20407. Ref T13272. This updates the "add panel" (which has two flavors: "add existing" and "create new") and "remove panel" flows to work with the new duplicate-friendly storage format.

We now modify panels by "panelKey", not by panel PHID, so one dashboard may have multiple copies of the same panel and we can still figure out what's going on.
We now work with "contextPHID", not "dashboardID", to make some flows with tab panels (or other nested panels in the future) easier.

The only major remaining flow is the Javascript "move panels around with drag-and-drop" flow.

Test Plan

Added panels to a dashboard with "Create New Panel".
Added panels to a dashboard with "Add Existing Panel".
Removed panels from a dashboard.
Added and removed duplicate panels, got a correctly-functioning dashboard that didn't care about duplicates.

Diff Detail

Repository

Branch

panel10

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 22568
Build 30910: Run Core Tests
Build 30909: arc lint + arc unit

Event Timeline

epriestley created this revision.Apr 12 2019, 6:29 PM

Harbormaster completed remote builds in B22568: Diff 48697.Apr 12 2019, 6:30 PM

epriestley requested review of this revision.Apr 12 2019, 6:30 PM

epriestley added a child revision: D20409: Make "Move Panel" on dashboards use the new storage and transactions.Apr 12 2019, 7:09 PM

amckinley accepted this revision.Apr 12 2019, 8:30 PM

amckinley added inline comments.

src/applications/auth/engine/PhabricatorAuthSessionEngine.php
424

This revision is now accepted and ready to land.Apr 12 2019, 8:30 PM

Closed by commit rP82c46f4b93bd: Update "add panel" and "remove panel" Dashboard flows to the new panel storage… (authored by epriestley). · Explain WhyApr 14 2019, 5:25 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

src/

__phutil_library_map__.php

applications/

auth/

engine/

PhabricatorAuthSessionEngine.php

4 lines

dashboard/

application/

PhabricatorDashboardApplication.php

5 lines

controller/

PhabricatorDashboardAddPanelController.php

PhabricatorDashboardRemovePanelController.php

dashboard/

PhabricatorDashboardAdjustController.php

202 lines

panel/

PhabricatorDashboardPanelEditController.php

38 lines

PhabricatorDashboardPanelRenderController.php

19 lines

editor/

PhabricatorDashboardPanelEditEngine.php

70 lines

engine/

PhabricatorDashboardPanelRenderingEngine.php

49 lines

PhabricatorDashboardRenderingEngine.php

28 lines

layoutconfig/

PhabricatorDashboardLayoutConfig.php

PhabricatorDashboardPanelRef.php

8 lines

PhabricatorDashboardPanelRefList.php

43 lines

storage/

PhabricatorDashboard.php

31 lines

xaction/

dashboard/

PhabricatorDashboardPanelsTransaction.php

153 lines

webroot/

rsrc/

js/

application/

dashboard/

behavior-dashboard-async-panel.js

3 lines

Diff 48697

resources/celerity/map.php

Loading...

src/__phutil_library_map__.php

Loading...

src/applications/auth/engine/PhabricatorAuthSessionEngine.php

Loading...

src/applications/dashboard/application/PhabricatorDashboardApplication.php

Loading...

src/applications/dashboard/controller/PhabricatorDashboardAddPanelController.php

Loading...

src/applications/dashboard/controller/PhabricatorDashboardRemovePanelController.php

Loading...

src/applications/dashboard/controller/dashboard/PhabricatorDashboardAdjustController.php

Loading...

src/applications/dashboard/controller/panel/PhabricatorDashboardPanelEditController.php

Loading...

src/applications/dashboard/controller/panel/PhabricatorDashboardPanelRenderController.php

Loading...

src/applications/dashboard/editor/PhabricatorDashboardPanelEditEngine.php

Loading...

src/applications/dashboard/engine/PhabricatorDashboardPanelRenderingEngine.php

Loading...

src/applications/dashboard/engine/PhabricatorDashboardRenderingEngine.php

Loading...

src/applications/dashboard/layoutconfig/PhabricatorDashboardLayoutConfig.php

Loading...

src/applications/dashboard/layoutconfig/PhabricatorDashboardPanelRef.php

Loading...

src/applications/dashboard/layoutconfig/PhabricatorDashboardPanelRefList.php

Loading...

src/applications/dashboard/storage/PhabricatorDashboard.php

Loading...

src/applications/dashboard/xaction/dashboard/PhabricatorDashboardPanelsTransaction.php

Loading...

webroot/rsrc/js/application/dashboard/behavior-dashboard-async-panel.js

Loading...