Skip to content

Commit

Permalink
Display alert about unset two-factor auth as warning, not as error
Browse files Browse the repository at this point in the history
  • Loading branch information
doobry-systemli committed Jul 17, 2022
1 parent bc03aab commit c033826
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion default_translations/en/messages.en.yml
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ form:
account:
twofactor:
headline: Two-factor authentication
unset: You don't have a two-factor authentication enabled yet.
unset: You don't have two-factor authentication enabled yet.
unset-extra: You need a TOTP app on your mobile device to enable two-factor authentication.
set: Two-factor authentication is enabled.
button: Configure two-factor authentication
Expand Down
4 changes: 2 additions & 2 deletions templates/Start/twofactor.html.twig
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
<h3>{{ "account.twofactor.headline"|trans }}</h3>
<p>{{ "account.twofactor.lead"|trans }}</p>
{% if not twofactor_enabled %}
<p class="alert alert-danger">
<i class="glyphicon glyphicon-alert" aria-hidden="true"></i>
<p class="alert alert-warning">
<i class="glyphicon glyphicon-warning-sign" aria-hidden="true"></i>
{{ "account.twofactor.unset"|trans }}
</p>
{% endif %}
Expand Down
2 changes: 1 addition & 1 deletion templates/User/twofactor.html.twig
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
{% else %}
{% if not twofactor_enabled %}
<div class="alert alert-warning">
<i class="glyphicon glyphicon-info-sign" aria-hidden="true"></i>
<i class="glyphicon glyphicon-warning-sign" aria-hidden="true"></i>
{{ "account.twofactor.unset"|trans }}
{{ "account.twofactor.unset-extra"|trans }}
</div>
Expand Down

0 comments on commit c033826

Please sign in to comment.