Page MenuHomePhabricator

Add an ApplicationProfilePanel
ClosedPublic

Authored by chad on Dec 9 2016, 7:10 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 7, 7:59 AM
Unknown Object (File)
Thu, Mar 28, 3:46 AM
Unknown Object (File)
Mon, Mar 25, 7:36 AM
Unknown Object (File)
Mon, Mar 25, 7:36 AM
Unknown Object (File)
Mon, Mar 25, 7:36 AM
Unknown Object (File)
Mon, Mar 25, 7:35 AM
Unknown Object (File)
Feb 16 2024, 9:29 PM
Unknown Object (File)
Feb 16 2024, 9:27 PM
Subscribers

Details

Summary

Allows applications to be added as profile menu items

Test Plan

Add an application to a project, see menu item, click on menu. Uninstall application, see menu without application.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Add an ApplicationProfilePanel.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.
  • If I uninstall the application, then click "Edit Menu", do I still see something sensible?
  • You could optionally let users rename these by adding another field to buildEditEngineFields(), so they can call "Maniphest" something like "Tasks" if they want.
src/applications/search/profilepanel/PhabricatorApplicationProfilePanel.php
40–49

I think it would be slightly better to use PhabricatorApplicationQuery with withPHIDs(...) here. In practice, it does the same thing, but we could optimize that more easily in the future if it ever shows up on a profile.

This revision is now accepted and ready to land.Dec 9 2016, 7:45 PM
chad edited edge metadata.
  • use PhabricatorApplicationQuery
  • Fake a name when uninstalled
This revision was automatically updated to reflect the committed changes.