Page MenuHomePhabricator

Add an ApplicationProfilePanel
ClosedPublic

Authored by chad on Dec 9 2016, 7:10 PM.
Tags
None
Referenced Files
F14015448: D17016.id40944.diff
Sun, Nov 3, 6:56 PM
F14004400: D17016.id40945.diff
Sat, Oct 26, 7:58 PM
F13999515: D17016.diff
Thu, Oct 24, 3:18 PM
F13979336: D17016.id40940.diff
Oct 19 2024, 4:04 AM
F13967738: D17016.id40940.diff
Oct 16 2024, 3:35 PM
F13956231: D17016.diff
Oct 14 2024, 4:50 AM
Unknown Object (File)
Oct 3 2024, 5:54 AM
Unknown Object (File)
Oct 3 2024, 5:54 AM
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.