don't strip iframe url if target is html (#3207)

Author: gabestein

workers/tasks/export/export.ts

@@ -35,11 +35,13 @@ export const exportTask = async ({ exportId }: { exportId: string }) => {
 			notesData,
 		});
 	} else {
-		const staticHtml = await renderStaticHtml({
+		const staticHtmlOptions = {
 			pubDoc,
 			notesData,
 			pubMetadata,
-		});
+			pagedTarget,
+		};
+		const staticHtml = await renderStaticHtml(staticHtmlOptions);
 		if (pagedTarget) {
 			url = (await exportWithPaged(staticHtml)).url;
 		} else {

workers/tasks/export/html.tsx

@@ -273,10 +273,11 @@ type RenderStaticHtmlOptions = {
 	pubDoc: DocJson;
 	pubMetadata: PubMetadata;
 	notesData: NotesData;
+	pagedTarget: boolean;
 };
 
 export const renderStaticHtml = async (options: RenderStaticHtmlOptions) => {
-	const { pubDoc, pubMetadata, notesData } = options;
+	const { pubDoc, pubMetadata, notesData, pagedTarget } = options;
 	const { title, nodeLabels, citationInlineStyle } = pubMetadata;
 	const { footnotes, citations, noteManager, renderedStructuredValues } = notesData;
 
@@ -287,9 +288,13 @@ export const renderStaticHtml = async (options: RenderStaticHtmlOptions) => {
 		renderedStructuredValues,
 	});
 
-	const renderableNodes = [filterNonExportableNodes, addHrefsToNotes, blankIframes]
-		.filter((x): x is (nodes: any) => any => !!x)
-		.reduce((nodes, fn) => fn(nodes), pubDoc.content);
+	const renderableNodes = pagedTarget
+		? [filterNonExportableNodes, addHrefsToNotes, blankIframes]
+				.filter((x): x is (nodes: any) => any => !!x)
+				.reduce((nodes, fn) => fn(nodes), pubDoc.content)
+		: [filterNonExportableNodes, addHrefsToNotes]
+				.filter((x): x is (nodes: any) => any => !!x)
+				.reduce((nodes, fn) => fn(nodes), pubDoc.content);
 
 	const docContent = renderStatic({
 		schema: editorSchema,