Skip to content

Commit 03d323e

Browse files
fooishbarfooishbar
authored and
fooishbar
committedSep 20, 2016
Fix config-migration text for dashboard options
Summary: The commit which added checks for the old homepage options (now in Dashboard) in rP9d9a47e9cf, added them to the auth section, where they would present: This option has been migrated to the "Auth" application. Your old configuration is still in effect, but now stored in "Auth" instead of configuration. Going forward, you can manage authentication from the web UI. Remove them from the moved-to-Auth list, and coalesce the multiple definitions of the help text into one. Test Plan: - set maniphest.priorities.unbreak-now to something - observe the setup issue reported - hope it tells you the right thing Reviewers: epriestley, #blessed_reviewers Reviewed By: epriestley, #blessed_reviewers Subscribers: Korvin, epriestley, chad Differential Revision: https://secure.phabricator.com/D16576
1 parent 51f8ec4 commit 03d323e

File tree

1 file changed

+7
-14
lines changed

1 file changed

+7
-14
lines changed
 

‎src/applications/config/check/PhabricatorExtraConfigSetupCheck.php

+7-14
Original file line numberDiff line numberDiff line change
@@ -143,9 +143,6 @@ public static function getAncientConfig() {
143143
'phabricator.auth-permanent',
144144
'phabricator.application-id',
145145
'phabricator.application-secret',
146-
'maniphest.priorities.unbreak-now',
147-
'maniphest.priorities.needs-triage',
148-
'welcome.html',
149146
);
150147

151148
$ancient_config = array_fill_keys($auth_config, $reason_auth);
@@ -197,6 +194,10 @@ public static function getAncientConfig() {
197194
'The "Re: Prefix" and "Vary Subjects" settings are now configured '.
198195
'in global settings.');
199196

197+
$dashboard_reason = pht(
198+
'This option has been removed, you can use Dashboards to provide '.
199+
'homepage customization. See T11533 for more details.');
200+
200201
$ancient_config += array(
201202
'phid.external-loaders' =>
202203
pht(
@@ -336,17 +337,9 @@ public static function getAncientConfig() {
336337
'This option has been replaced with `ui.logo`, which provides more '.
337338
'flexible configuration options.'),
338339

339-
'welcome.html' => pht(
340-
'This option has been removed, you can use Dashboards to provide '.
341-
'homepage customization. See T11533 for more details.'),
342-
343-
'maniphest.priorities.unbreak-now' => pht(
344-
'This option has been removed, you can use Dashboards to provide '.
345-
'homepage customization. See T11533 for more details.'),
346-
347-
'maniphest.priorities.needs-triage' => pht(
348-
'This option has been removed, you can use Dashboards to provide '.
349-
'homepage customization. See T11533 for more details.'),
340+
'welcome.html' => $dashboard_reason,
341+
'maniphest.priorities.unbreak-now' => $dashboard_reason,
342+
'maniphest.priorities.needs-triage' => $dashboard_reason,
350343

351344
);
352345

0 commit comments

Comments
 (0)
Failed to load comments.