docs: add todos for next steps

guardian · Sep 11, 2023 · 0528f48 · 0528f48
1 parent dfc452d
commit 0528f48
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/dotcom-rendering/src/model/enhanceBlocks.ts b/dotcom-rendering/src/model/enhanceBlocks.ts
@@ -96,6 +96,9 @@ type Options = {
 	promotedNewsletter: Newsletter | undefined;
 };
 
+// TODO - add consideration of rendering target / platform
+// so that we can add app-only enhancers
+
 // IMPORTANT: the ordering of the enhancer is IMPORTANT to keep in mind
 // example: enhanceInteractiveContentElements needs to be before enhanceNumberedLists
 // as they both effect SubheadingBlockElement

diff --git a/dotcom-rendering/src/server/index.article.apps.ts b/dotcom-rendering/src/server/index.article.apps.ts
@@ -6,6 +6,9 @@ import { renderArticle } from './render.article.apps';
 
 export const handleAppsArticle: RequestHandler = ({ body }, res) => {
 	recordTypeAndPlatform('article', 'apps');
+
+	// TODO - add renderingTarget so that we can enhance
+	// blocks specifically for Apps
 	const article = enhanceArticleType(body);
 	const { html, prefetchScripts } = renderArticle(article);