Page MenuHomePhabricator

Make the "Install Dashboard" flow smoother
ClosedPublic

Authored by epriestley on Apr 1 2019, 8:10 PM.
Tags
None
Referenced Files
F13055908: D20364.id48619.diff
Fri, Apr 19, 1:03 PM
F13050694: D20364.diff
Fri, Apr 19, 3:07 AM
Unknown Object (File)
Wed, Apr 17, 7:53 PM
Unknown Object (File)
Sat, Apr 6, 9:22 AM
Unknown Object (File)
Sun, Mar 31, 12:58 PM
Unknown Object (File)
Fri, Mar 29, 7:22 AM
Unknown Object (File)
Mar 10 2024, 4:26 AM
Unknown Object (File)
Feb 7 2024, 1:20 PM
Subscribers
None

Details

Summary

Depends on D20362. Ref T13272. Currently, Dashboards have an "Install Dashboard" flow which is pretty janky and only allows you to install things to the home page.

Instead, allow users to install things to any valid target (home, favorites, portals, projects). This also provides URIs like dashboard/install/1/home/personal/ which allow you to link users to an "install a dashboard" page; this may or may not get used.

Test Plan

Installed dashboards on home, favorites, projects, and portals.

Diff Detail

Repository
rP Phabricator
Branch
portal13
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22457
Build 30743: Run Core Tests
Build 30742: arc lint + arc unit

Event Timeline

amckinley added inline comments.
src/applications/search/menuitem/PhabricatorProfileMenuItem.php
91–92

Debugging code.

This revision is now accepted and ready to land.Apr 3 2019, 6:37 PM
This revision was automatically updated to reflect the committed changes.