Homestyx hydra
Diffusion hydra 89dcbe125c6c

Panels list: fix missing welcome page

Description

Panels list: fix missing welcome page

Summary:
The method getNewUserBody() was never designed for a list controller.
The method was just orphan. Now, instead, it's non-orphan, and it works
as expected.

This bug was highlighted by Andre Klapper, since their linter was screaming
about inconsistences in the current nonsense usage of getNewUserBody():

D25649

The code was just moved from the list controller, to the search engine.

  • PhabricatorDashboardPanelListController (from)
  • PhabricatorDashboardPanelSearchEngine (to)

Bonus point:

Adjusted the icon and the title to talk about "Panels" and not "Dashboards".

Added also some inline documentation with an hint about ?nux=1.
So now we can easily remember how to test this kind of things.

Closes T15844

Test Plan:
In both these pages, you finally see a welcome message, not just in Maniphest:

In both these pages, if you have no elements, you finally see a welcome message,
and not just in Maniphest:

Check that the welcome message helps you in creating a new Panel, and that
talks about Panels, and that have the lovely icon of a Panel.

Reviewers: aklapper, O1 Blessed Committers

Reviewed By: aklapper, O1 Blessed Committers

Subscribers: tobiaswiese, Matthew, Cigaryno

Maniphest Tasks: T15844

Differential Revision: https://we.phorge.it/D25674

Details

Provenance
Valerio BozzolanAuthored on Jun 3 2024, 7:44 PM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:40720ab83d01: Expose initially set values of Maniphest custom date field and point values…
Branches
Unknown
Tags
Unknown

Event Timeline