AI Title Optimization: Add error handling for Title Optimization (#37195

) * AI Title Optimization: Handle errors * changelog * AI Title Optimization: Disable entry button if no post content * AI Title Optimization: Rephrasing error message Co-authored-by: Douglas Henri <[email protected]> --------- Co-authored-by: Douglas Henri <[email protected]>
Automattic · May 9, 2024 · 64b82ca · 64b82ca
1 parent 656b6f6
commit 64b82ca
Show file tree

Hide file tree

Showing 3 changed files with 46 additions and 17 deletions.
diff --git a/projects/plugins/jetpack/changelog/add-title-optimization-error b/projects/plugins/jetpack/changelog/add-title-optimization-error
@@ -0,0 +1,4 @@
+Significance: minor
+Type: other
+
+Add error handling on Title Optimization
diff --git a/...ns/jetpack/extensions/plugins/ai-assistant-plugin/components/title-optimization/index.tsx b/...ns/jetpack/extensions/plugins/ai-assistant-plugin/components/title-optimization/index.tsx
@@ -32,6 +32,7 @@ export default function TitleOptimization( {
 	const [ isTitleOptimizationModalVisible, setIsTitleOptimizationModalVisible ] = useState( false );
 	const [ generating, setGenerating ] = useState( false );
 	const [ options, setOptions ] = useState( [] );
+	const [ error, setError ] = useState( false );
 	const { editPost } = useDispatch( 'core/editor' );
 	const { autosave } = useAutoSaveAndRedirect();
 	const { increaseAiAssistantRequestsCount } = useDispatch( 'wordpress-com/plans' );
@@ -61,6 +62,7 @@ export default function TitleOptimization( {
 	const { request, stopSuggestion } = useAiSuggestions( {
 		onDone: handleDone,
 		onError: () => {
+			setError( true );
 			setGenerating( false );
 		},
 	} );
@@ -91,6 +93,12 @@ export default function TitleOptimization( {
 		handleRequest();
 	}, [ handleRequest, placement, recordEvent, toggleTitleOptimizationModal ] );
 
+	const handleTryAgain = useCallback( () => {
+		setError( false );
+		setGenerating( true );
+		handleRequest();
+	}, [ handleRequest ] );
+
 	const handleAccept = useCallback(
 		( event: MouseEvent ) => {
 			// track the generated title acceptance
@@ -120,7 +128,7 @@ export default function TitleOptimization( {
 			<p>{ __( 'Use AI to optimize key details of your post.', 'jetpack' ) }</p>
 			<Button
 				isBusy={ busy }
-				disabled={ disabled }
+				disabled={ ! postContent || disabled }
 				onClick={ handleTitleOptimization }
 				variant="secondary"
 			>
@@ -140,25 +148,42 @@ export default function TitleOptimization( {
 						</div>
 					) : (
 						<>
-							<span className="jetpack-ai-title-optimization__intro">
-								{ __( 'Choose an optimized title below:', 'jetpack' ) }
-							</span>
-							<TitleOptimizationOptions
-								onChangeValue={ e => setSelected( e.target.value ) }
-								selected={ selected }
-								options={ options?.map?.( option => ( {
-									value: option.title,
-									label: option.title,
-									description: option.explanation,
-								} ) ) }
-							/>
+							{ error ? (
+								<div className="jetpack-ai-title-optimization__error">
+									{ __(
+										'The generation of your suggested titles failed. Please try again.',
+										'jetpack'
+									) }
+								</div>
+							) : (
+								<>
+									<span className="jetpack-ai-title-optimization__intro">
+										{ __( 'Choose an optimized title below:', 'jetpack' ) }
+									</span>
+									<TitleOptimizationOptions
+										onChangeValue={ e => setSelected( e.target.value ) }
+										selected={ selected }
+										options={ options?.map?.( option => ( {
+											value: option.title,
+											label: option.title,
+											description: option.explanation,
+										} ) ) }
+									/>
+								</>
+							) }
 							<div className="jetpack-ai-title-optimization__cta">
 								<Button variant="secondary" onClick={ toggleTitleOptimizationModal }>
 									{ __( 'Cancel', 'jetpack' ) }
 								</Button>
-								<Button variant="primary" onClick={ handleAccept }>
-									{ __( 'Replace title', 'jetpack' ) }
-								</Button>
+								{ error ? (
+									<Button variant="primary" onClick={ handleTryAgain }>
+										{ __( 'Try again', 'jetpack' ) }
+									</Button>
+								) : (
+									<Button variant="primary" onClick={ handleAccept }>
+										{ __( 'Replace title', 'jetpack' ) }
+									</Button>
+								) }
 							</div>
 						</>
 					) }

diff --git a/...s/jetpack/extensions/plugins/ai-assistant-plugin/components/title-optimization/style.scss b/...s/jetpack/extensions/plugins/ai-assistant-plugin/components/title-optimization/style.scss
@@ -9,7 +9,7 @@
 		justify-content: center;
 	}
 
-	&__intro {
+	&__intro, &__error {
 		margin-top: 16px;
 	}