revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-23 05:50:55 +01:00
Code Issues Releases Wiki Activity

Mark "Settings" and "Nuance" as launchable applications

Browse source 
Summary:
Fixes T12790. I don't think this was actually a regression, Settings just wasn't launchable before global settings (since it had no real landing page, and the profile menu always had a link) and didn't get marked launchable once we added them.

I also double-checked other un-launchable apps; Nuance is probably close enough to make launchable now while I'm in here.

Test Plan: Typed "settings" into global typeahead, got settings.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12790

Differential Revision: https://secure.phabricator.com/D18062
This commit is contained in:
epriestley 2017-06-01 12:31:22 -07:00
parent 0d8aba8550
commit b9a4988df3
2 changed files with 0 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/applications/nuance/application/PhabricatorNuanceApplication.php
View file

@ -18,11 +18,6 @@ final class PhabricatorNuanceApplication extends PhabricatorApplication {
return true;
}
public function isLaunchable() {
// Try to hide this even more for now.
return false;
}
public function getBaseURI() {
return '/nuance/';
}

4
src/applications/settings/application/PhabricatorSettingsApplication.php
View file

@ -22,10 +22,6 @@ final class PhabricatorSettingsApplication extends PhabricatorApplication {
return false;
}
public function isLaunchable() {
return false;
}
public function getRoutes() {
$panel_pattern = '(?:page/(?P<pageKey>[^/]+)/(?:(?P<formSaved>saved)/)?)?';