Page MenuHomePhabricator

Move Dashboard panel controllers into a "panel/" subdirectory
ClosedPublic

Authored by epriestley on Apr 2 2019, 4:17 PM.
Tags
None
Referenced Files
F14056443: D20369.diff
Sat, Nov 16, 8:38 PM
F14006221: D20369.id48635.diff
Mon, Oct 28, 3:30 AM
F14002206: D20369.diff
Fri, Oct 25, 4:19 PM
F13977938: D20369.diff
Oct 18 2024, 8:43 PM
F13966124: D20369.id48594.diff
Oct 16 2024, 6:21 AM
F13956928: D20369.id.diff
Oct 14 2024, 7:35 AM
Unknown Object (File)
Oct 9 2024, 11:59 AM
Unknown Object (File)
Oct 1 2024, 9:52 AM
Subscribers
None

Details

Summary

Depends on D20368. Ref T13272. Dashboards have a lot of controllers, try to organize them a little better.

Note "EditController" vs "EditproController". Yikes.

Test Plan

Loaded dashboards. No code changes.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Wait, we have both of PhabricatorDashboardPanelEditproController and PhabricatorDashboardPanelEditController, and they don't have the same code, and we're keeping both of them? Did you write one, forget about it, couldn't find it because of the incorrect name, and write it again?

This revision is now accepted and ready to land.Apr 3 2019, 6:50 PM

Ok this makes more sense now that I've looked at the follow up revisions.

I think the backstory was that Editpro got built out just enough to support a *.edit API for T10855, but finishing it would have been a much larger amount of work (i.e., the work I'm doing now) so I just paused us in a sketchy-but-stable state with both controllers in existence. The plan was always for Editpro to replace Edit some day (and that day is D20371).

This revision was automatically updated to reflect the committed changes.