Skip to content

Commit e5e0643

Browse files
authored
fix(sso): change Jetpack configuration notice from error to warning (#4383)
1 parent 3d5b8d5 commit e5e0643

File tree

1 file changed

+12
-14
lines changed

1 file changed

+12
-14
lines changed

src/wizards/newspack/views/settings/connections/jetpack-sso.tsx

Lines changed: 12 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
* WordPress dependencies
33
*/
44
import { __ } from '@wordpress/i18n';
5-
import { BaseControl, CheckboxControl } from '@wordpress/components';
5+
import { BaseControl, CheckboxControl, ExternalLink } from '@wordpress/components';
66
import { useEffect, useState } from '@wordpress/element';
77
import apiFetch from '@wordpress/api-fetch';
88

@@ -63,13 +63,9 @@ const JetpackSSO = () => {
6363
<ActionCard
6464
isMedium
6565
title={ __( 'Force two-factor authentication', 'newspack-plugin' ) }
66-
description={ () => (
67-
<>
68-
{ __(
69-
'Improve security by requiring two-factor authentication via WordPress.com for users with higher capabilities.',
70-
'newspack-plugin'
71-
) }
72-
</>
66+
description={ __(
67+
'Improve security by requiring two-factor authentication via WordPress.com for users with higher capabilities.',
68+
'newspack-plugin'
7369
) }
7470
hasGreyHeader={ !! settings.force_2fa }
7571
toggleChecked={ !! settings.force_2fa }
@@ -92,16 +88,18 @@ const JetpackSSO = () => {
9288
{ error && <Notice isError noticeText={ error } /> }
9389
{ settings.jetpack_sso_force_2fa && (
9490
<>
95-
<Notice
96-
isError
97-
noticeText={ __(
91+
<Notice isWarning>
92+
{ __(
9893
'Two-factor authentication is currently enforced for all users via Jetpack configuration.',
9994
'newspack-plugin'
100-
) }
101-
/>
95+
) }{ ' ' }
96+
<ExternalLink href="/wp-admin/admin.php?page=jetpack#/settings">
97+
{ __( 'Jetpack Settings', 'newspack-plugin' ) }
98+
</ExternalLink>
99+
</Notice>
102100
<p>
103101
{ __(
104-
'Customize which capabilties to enforce 2FA by untoggling the “Require accounts to use WordPress.com Two-Step Authentication” option in Jetpack settings.',
102+
'Customize which capabilities to enforce 2FA by untoggling the “Require accounts to use WordPress.com Two-Step Authentication” option in Jetpack settings.',
105103
'newspack-plugin'
106104
) }
107105
</p>

0 commit comments

Comments
 (0)