Newsletter: Email Menu: Don't prompt for connection on simple sites. (#…

…39064)
Automattic · Aug 25, 2024 · 18a6609 · 18a6609
1 parent d2a46b6
commit 18a6609
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/projects/plugins/jetpack/changelog/fix-simple-sites-prompts-connection b/projects/plugins/jetpack/changelog/fix-simple-sites-prompts-connection
@@ -0,0 +1,4 @@
+Significance: patch
+Type: other
+
+wpcom only
diff --git a/projects/plugins/jetpack/extensions/blocks/subscriptions/menu.js b/projects/plugins/jetpack/extensions/blocks/subscriptions/menu.js
@@ -1,4 +1,5 @@
 import { useConnection } from '@automattic/jetpack-connection';
+import { isSimpleSite } from '@automattic/jetpack-shared-extension-utils';
 import { Button, PanelBody, __experimentalHStack as HStack } from '@wordpress/components'; // eslint-disable-line @wordpress/no-unsafe-wp-apis
 import { useSelect } from '@wordpress/data';
 import { PluginSidebar } from '@wordpress/edit-post';
@@ -31,6 +32,7 @@ const NewsletterMenu = () => {
 
 	const { isUserConnected } = useConnection();
 	const connectUrl = `${ window?.Jetpack_Editor_Initial_State?.adminUrl }admin.php?page=my-jetpack#/connection`;
+	const shouldPromptForConnection = ! isSimpleSite() && ! isUserConnected;
 
 	const openPreviewModal = () => setIsPreviewModalOpen( true );
 	const closePreviewModal = () => setIsPreviewModalOpen( false );
@@ -48,7 +50,7 @@ const NewsletterMenu = () => {
 				<SubscribersAffirmation accessLevel={ accessLevel } prePublish={ ! isPublished } />
 				{ isSendEmailEnabled && ! isPublished && (
 					<>
-						{ isUserConnected ? (
+						{ ! shouldPromptForConnection ? (
 							<>
 								<p>
 									{ __(