revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-19 13:22:42 +01:00
Code Issues Releases Wiki Activity

Config - make "No auth providers configured" check use proper hyperlinking technology

Browse source 
Summary: Fixes T8346. Also gets rid of the air quotes on Auth and uses the "Appname Application" style we use elsewhere.

Test Plan: slapped an "|| true" in the checking logic and verified the link was rendered in a usable fashion just below the error message.

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: epriestley, Korvin

Maniphest Tasks: T8346

Differential Revision: https://secure.phabricator.com/D13315
This commit is contained in:
Bob Trahan 2015-06-16 14:49:52 -07:00
parent 281082f4d7
commit ca8adac1eb
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/applications/config/check/PhabricatorAuthSetupCheck.php
View file

@ -27,14 +27,14 @@ final class PhabricatorAuthSetupCheck extends PhabricatorSetupCheck {
'You have not configured any authentication providers yet. You '. 'You have not configured any authentication providers yet. You '.
'should add a provider (like username/password, LDAP, or GitHub '. 'should add a provider (like username/password, LDAP, or GitHub '.
'OAuth) so users can register and log in. You can add and configure '. 'OAuth) so users can register and log in. You can add and configure '.
'providers using the [[%s | "Auth" application]].', 'providers using the Auth Application.');
'/auth/');
$this $this
->newIssue('auth.noproviders') ->newIssue('auth.noproviders')
->setShortName(pht('No Auth Providers')) ->setShortName(pht('No Auth Providers'))
->setName(pht('No Authentication Providers Configured')) ->setName(pht('No Authentication Providers Configured'))
->setMessage($message); ->setMessage($message)
->addLink('/auth/', pht('Auth Application'));
} }
} }
} }