@griddo/ax 12.10.3 → 12.10.5-rc.0

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@griddo/ax",
 	"description": "Griddo Author Experience",
-	"version": "12.10.3",
+	"version": "12.10.5-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": "2d015ededd3170ca82057b809faaafc6b91c80ff"
+	"gitHead": "1b249444fef8e63a26ee518b8d49c2939b2d3992"
 }

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

@@ -87,6 +87,23 @@ describe("ConditionalField component rendering", () => {
 	});
 });
 
+describe("ConditionalField spacing", () => {
+	it("should reset the bottom margin of its direct child only", () => {
+		render(
+			<ThemeProvider theme={parseTheme(globalTheme)}>
+				<ConditionalField {...defaultProps} />
+			</ThemeProvider>,
+		);
+
+		const conditionalFieldContent = screen.getByTestId("conditionalFieldContent");
+
+		expect(conditionalFieldContent).toHaveStyleRule("margin-bottom", "0", { modifier: "> div:last-child" });
+		// A descendant selector would also reach every nested field and "Add" button,
+		// flattening the spacing of nested ArrayFieldGroups (sc-110266).
+		expect(conditionalFieldContent).not.toHaveStyleRule("margin-bottom", "0", { modifier: "div:last-child" });
+	});
+});
+
 interface IConditionalFieldProps {
 	value: string | boolean;
 	options: any[];

package/src/__tests__/modules/Content/NewContentModal/PageImporter.test.tsx

@@ -0,0 +1,110 @@
+import { parseTheme } from "@ax/helpers";
+import PageImporter from "@ax/modules/Content/NewContentModal/PageImporter";
+import globalTheme from "@ax/themes/theme.json";
+
+import configureStore from "redux-mock-store";
+import thunk from "redux-thunk";
+import { ThemeProvider } from "styled-components";
+
+import { act, cleanup, fireEvent, render, screen } from "./../../../../../config/tests/test-utils";
+
+const getGlobalPagesLight = vi.fn();
+
+vi.mock("@ax/api", async (importOriginal) => {
+	const actual = (await importOriginal()) as any;
+
+	return {
+		...actual,
+		sites: {
+			...actual.sites,
+			getGlobalPagesLight: (...args: any[]) => getGlobalPagesLight(...args),
+		},
+	};
+});
+
+afterEach(cleanup);
+
+const buildPage = (id: number) => ({
+	id,
+	title: `insti news ${id}`,
+	slug: `/insti-news-${id}`,
+	modified: "2026-09-22T10:00:00.000Z",
+});
+
+const setPagesToImport = vi.fn();
+
+const renderImporter = async () => {
+	const store = configureStore([thunk])({
+		app: { lang: { id: 4, locale: "en_GB" } },
+		sites: { currentSiteInfo: { id: 1 } },
+	});
+
+	await act(async () => {
+		render(
+			<ThemeProvider theme={parseTheme(globalTheme)}>
+				<PageImporter
+					structuredDataID="INS_NEWS"
+					title="Get Institutional News from Global"
+					setPagesToImport={setPagesToImport}
+					toggleImporter={vi.fn()}
+					importPage={vi.fn()}
+				/>
+			</ThemeProvider>,
+			{ store },
+		);
+	});
+};
+
+const search = async (query: string) => {
+	const input = screen.getByTestId("search-input");
+	await act(async () => {
+		fireEvent.change(input, { target: { value: query } });
+	});
+	await act(async () => {
+		fireEvent.keyDown(input, { key: "Enter" });
+	});
+};
+
+const selectPage = async (pageId: number) => {
+	const checkbox = screen
+		.getAllByTestId("check-field-input")
+		.find((input) => input.getAttribute("value") === String(pageId)) as HTMLElement;
+
+	await act(async () => {
+		fireEvent.click(checkbox);
+	});
+};
+
+const addPagesButton = () => screen.getByTestId("button-default");
+
+describe("[UC-AX-explore-manage-pages] PageImporter — importar páginas globales", () => {
+	beforeEach(() => {
+		// El listado inicial trae una primera tanda; la búsqueda devuelve una página
+		// que no venía en ella, que es justo lo que hace el backend al filtrar por query.
+		getGlobalPagesLight.mockImplementation(async ({ query }: { query: string }) => ({
+			status: 200,
+			data: { items: query ? [buildPage(3)] : [buildPage(1), buildPage(2)] },
+		}));
+	});
+
+	it("habilita 'Add pages' con una página de la primera tanda", async () => {
+		await renderImporter();
+
+		await selectPage(1);
+
+		expect(addPagesButton()).not.toBeDisabled();
+	});
+
+	it("habilita 'Add pages' con una página que solo aparece al buscar", async () => {
+		await renderImporter();
+
+		await search("insti news 3");
+		await selectPage(3);
+
+		expect(addPagesButton()).not.toBeDisabled();
+		// Y la columna de la derecha la lista, en vez de seguir pidiendo una selección.
+		expect(screen.queryByText(/You have to select at least one page/)).not.toBeInTheDocument();
+		// El conjunto que se importa es el seleccionado, no el de la primera tanda.
+		expect(setPagesToImport).toHaveBeenLastCalledWith([3]);
+	});
+});

package/src/components/Fields/ConditionalField/style.tsx

@@ -7,7 +7,7 @@ export const Wrapper = styled.div`
 export const Content = styled.div`
   transition: all 0.5s ease-in-out;
   margin-top: ${(p) => p.theme.spacing.m};
-  div:last-child {
+  > div:last-child {
     margin-bottom: 0;
   }
 `;

package/src/modules/Content/NewContentModal/PageImporter/index.tsx

@@ -48,7 +48,14 @@ const PageImporter = (props: IPageImporterProps) => {
 					data: { items },
 				} = response;
 				setPages(items);
-				if (!allPages) setAllPages(items);
+				// `allPages` acumula todo lo que se ha listado, no solo la primera tanda: la
+				// columna de seleccionadas y el botón "Add pages" tienen que poder resolver
+				// una página que apareció por primera vez en una búsqueda posterior.
+				setAllPages((prevPages) => {
+					const knownPages = prevPages ?? [];
+					const newPages = items.filter((item: IPage) => !knownPages.some((page) => page.id === item.id));
+					return newPages.length ? [...knownPages, ...newPages] : knownPages;
+				});
 			}
 			setIsLoading(false);
 		} catch (error) {