@griddo/ax 12.8.6 → 12.8.8-rc.0

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@griddo/ax",
 	"description": "Griddo Author Experience",
-	"version": "12.8.6",
+	"version": "12.8.8-rc.0",
 	"authors": [
 		"Álvaro Sánchez' <alvaro.sanches@secuoyas.com>",
 		"Diego M. Béjar <diego.bejar@secuoyas.com>",
@@ -201,5 +201,5 @@
 	"publishConfig": {
 		"access": "public"
 	},
-	"gitHead": "59241d09ab7aef8ffbe26b74a2095b05dae6faee"
+	"gitHead": "663a3900aff18833cb1fe18f8854f0d42fad2a57"
 }

package/src/__tests__/components/Fields/TranslateButton/TranslateButton.test.tsx

@@ -157,10 +157,16 @@ describe("Translate Button component rendering", () => {
 		const translateButtonWrapper = screen.queryByTestId("translate-button-wrapper");
 		const translatedNotificationWrapper = screen.queryByTestId("update-translation-button-wrapper");
 		expect(translatedNotificationWrapper).toBeTruthy();
+		expect(translatedNotificationWrapper?.textContent).toContain("You can update it");
+		expect(screen.queryByText("Update translation with AI")).toBeTruthy();
 		expect(translateButtonWrapper).toBeFalsy();
 	});
 
-	it("should render the notification if structured data is already translated", () => {
+	// sc-119164 — la retraducción solo existe para páginas. El aviso se mostraba también en
+	// datos y formularios, y al pulsarlo no se llamaba a ninguna API: `getRetranslationAction`
+	// es undefined ahí, así que el componente pintaba «The translation could not be completed»
+	// sin haber intentado nada. Mientras no haya endpoint, no se ofrece la acción.
+	it("should show the already-translated notice for structured data but NOT offer the update action", () => {
 		const initialStore = {
 			app: { globalSettings: { autoTranslation: true } },
 			pageEditor: { editorContent: null, isIATranslated: false },
@@ -178,10 +184,35 @@ describe("Translate Button component rendering", () => {
 			{ store },
 		);
 
-		const translateButtonWrapper = screen.queryByTestId("translate-button-wrapper");
-		const translatedNotificationWrapper = screen.queryByTestId("update-translation-button-wrapper");
-		expect(translatedNotificationWrapper).toBeTruthy();
-		expect(translateButtonWrapper).toBeFalsy();
+		const translatedNotificationWrapper = screen.getByTestId("update-translation-button-wrapper");
+		expect(translatedNotificationWrapper.textContent).toContain("already been translated");
+		expect(translatedNotificationWrapper.textContent).not.toContain("You can update it");
+		expect(screen.queryByText("Update translation with AI")).toBeFalsy();
+		expect(screen.queryByTestId("translate-button-wrapper")).toBeFalsy();
+	});
+
+	it("should show the already-translated notice for forms but NOT offer the update action", () => {
+		const initialStore = {
+			app: { globalSettings: { autoTranslation: true } },
+			pageEditor: { editorContent: null, isIATranslated: false },
+			structuredData: { form: null, isIATranslated: false },
+			forms: { formContent: { canBeTranslated: false, originalLanguage: 2 } },
+		};
+		const store = mockStore(initialStore);
+
+		defaultProps.contentType = "form";
+
+		render(
+			<ThemeProvider theme={parseTheme(globalTheme)}>
+				<TranslateButton {...defaultProps} />
+			</ThemeProvider>,
+			{ store },
+		);
+
+		const wrapper = screen.getByTestId("update-translation-button-wrapper");
+		expect(wrapper.textContent).toContain("already been translated");
+		expect(screen.queryByText("Update translation with AI")).toBeFalsy();
+		expect(screen.queryByTestId("translate-button-wrapper")).toBeFalsy();
 	});
 
 	it("should render the component if page is already translated but can be translated", () => {

package/src/components/Fields/TranslateButton/index.tsx

@@ -47,6 +47,14 @@ const TranslateButton = (props: ITranslateButtonProps): JSX.Element => {
 
 	const isOriginal = content?.originalLanguage === lang;
 
+	// La retraducción solo está implementada para páginas: `getContent` no define
+	// `getRetranslationAction` para datos ni formularios. Sin ella, el botón despachaba
+	// `undefined?.(...)` y pintaba «The translation could not be completed» sin haber llamado a
+	// ninguna API (sc-119164). Se mantiene el aviso de que el contenido ya está traducido, pero
+	// no se ofrece una acción inexistente. El día que Back añada el endpoint para datos o
+	// formularios, basta con registrar la acción en `getContent`.
+	const canRetranslate = !!getRetranslationAction;
+
 	const initialState = {
 		isLoading: false,
 		error: false,
@@ -88,42 +96,48 @@ const TranslateButton = (props: ITranslateButtonProps): JSX.Element => {
 				<>
 					<S.Wrapper data-testid="update-translation-button-wrapper" isSmall={!!structuredDataForm}>
 						<S.Text>
-							This page has <strong>already been translated</strong>. You can update it with the latest changes from the
-							source language.
+							This {canRetranslate ? "page" : "content"} has <strong>already been translated</strong>.
+							{canRetranslate && " You can update it with the latest changes from the source language."}
 						</S.Text>
-						<S.ButtonWrapper>
-							<S.StyledButton
-								type="button"
-								onClick={() => toggleModal("update")}
-								icon={!state.isLoading ? "Ia" : undefined}
-								loader={state.isLoading ? "circle" : undefined}
-								disabled={disabled}
-								buttonStyle="line"
-							>
-								{updateButtonText}
-							</S.StyledButton>
-						</S.ButtonWrapper>
-						{state.error && (
-							<S.ErrorText>
-								The translation could not be completed. Your previous content has been preserved.
-							</S.ErrorText>
+						{canRetranslate && (
+							<>
+								<S.ButtonWrapper>
+									<S.StyledButton
+										type="button"
+										onClick={() => toggleModal("update")}
+										icon={!state.isLoading ? "Ia" : undefined}
+										loader={state.isLoading ? "circle" : undefined}
+										disabled={disabled}
+										buttonStyle="line"
+									>
+										{updateButtonText}
+									</S.StyledButton>
+								</S.ButtonWrapper>
+								{state.error && (
+									<S.ErrorText>
+										The translation could not be completed. Your previous content has been preserved.
+									</S.ErrorText>
+								)}
+							</>
 						)}
 					</S.Wrapper>
-					<Modal
-						isOpen={isOpen("update")}
-						hide={() => toggleModal("update")}
-						size="S"
-						title="Update translation page with AI"
-						mainAction={{ title: "Update translation with AI", icon: "Ia", onClick: handleUpdateClick }}
-						secondaryAction={{ title: "Cancel", onClick: () => toggleModal("update") }}
-						height="auto"
-					>
-						<S.ModalContent>
-							This page <strong>already has a translation</strong>. Updating it{" "}
-							<strong>will replace all existing text in this language</strong>, including any manual edits made after
-							the previous translation. This action cannot be undone.
-						</S.ModalContent>
-					</Modal>
+					{canRetranslate && (
+						<Modal
+							isOpen={isOpen("update")}
+							hide={() => toggleModal("update")}
+							size="S"
+							title="Update translation page with AI"
+							mainAction={{ title: "Update translation with AI", icon: "Ia", onClick: handleUpdateClick }}
+							secondaryAction={{ title: "Cancel", onClick: () => toggleModal("update") }}
+							height="auto"
+						>
+							<S.ModalContent>
+								This page <strong>already has a translation</strong>. Updating it{" "}
+								<strong>will replace all existing text in this language</strong>, including any manual edits made after
+								the previous translation. This action cannot be undone.
+							</S.ModalContent>
+						</Modal>
+					)}
 				</>
 			)}
 			{autoTranslation && content?.canBeTranslated ? (