Page MenuHomePhabricator

Implement a "pro" EditEngine for dashboard panels
ClosedPublic

Authored by epriestley on Jul 3 2016, 9:47 PM.
Tags
None
Referenced Files
F12832287: D16226.id39031.diff
Thu, Mar 28, 1:18 PM
Unknown Object (File)
Fri, Mar 8, 7:16 PM
Unknown Object (File)
Fri, Mar 8, 7:16 PM
Unknown Object (File)
Fri, Mar 8, 7:16 PM
Unknown Object (File)
Thu, Feb 29, 12:40 PM
Unknown Object (File)
Feb 7 2024, 6:46 PM
Unknown Object (File)
Feb 3 2024, 11:14 AM
Unknown Object (File)
Feb 2 2024, 9:32 AM
Subscribers
None

Details

Summary

Ref T10855. This can't replace the old edit flow yet, but get the basics in place.

(This is actually much closer to just being able to swap than I anticipated since CustomFields sort of just work, but the exiting flow has some "clone existing panel" / "place directly on dashboard" stuff that this doesn't yet.)

Test Plan

Created and edited a panel by manually using the "editpro" flow.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Implement a "pro" EditEngine for dashboard panels.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

Can we get rid of projects on panels, too?

epriestley edited edge metadata.
  • Also just sketch in dashboard.panel.edit here.

Yes, but they seem possibly useful to me -- are you only concerned about T11161?

For panels other than Query panels (text, tabs, actions, phriction, etc. -- all other future panels?) it seems like this probably isn't confusing. Maybe the root issue is just "query panels are weird right now"?

That said, there's also no current use for it, so we can nuke it until one arises and then that use case can fight confusion on query panels, I guess.

Yeah I've never used it, and I can see Projects useful maybe for Dashboards, but not Panels so much. Both seem like a mess. to find anything useful.

chad edited edge metadata.
This revision is now accepted and ready to land.Jul 3 2016, 10:06 PM
epriestley edited edge metadata.
  • Remove ProjectInterface to get rid of "Tags" on EditProController.
  • Remove all the tag logic from EditController manually.

Once we swap over fully adding or removing support is just a matter of adding or removing implements PhabricatorProjectInterface, so starting simpler seems reasonable to me.

This revision was automatically updated to reflect the committed changes.