Merge pull request #34 from Financial-Times/component-mapping

- Add new components to components.js list. - Add "@financial-times/ftdomdelegate" exception to `createComponentName`.
Financial-Times · Nov 12, 2019 · 7ff3d8b · 7ff3d8b
2 parents fd7a881 + a798edb
commit 7ff3d8b
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/config/components.js b/config/components.js
@@ -58,5 +58,7 @@ export default [
 	"o-viewport",
 	"o-visual-effects",
 	"o-spacing",
-	"o-lazy-load"
+	"o-lazy-load",
+	"o-editorial-typography",
+	"o-editorial-layout",
 ]
diff --git a/lib/npm.js b/lib/npm.js
@@ -86,7 +86,13 @@ export let mergeManifests = (existing, generated) => {
 export let createComponentName = (
 	componentName,
 	npmOrganisation = settings.npmOrganisation
-) => `@${npmOrganisation}/${componentName}`
+) => {
+	// ftdomdelegate is special. It is not published under any org.
+	if (componentName == 'ftdomdelegate') {
+		return 'ftdomdelegate';
+	}
+	return `@${npmOrganisation}/${componentName}`;
+}
 
 /**
  * Get an npm package name for a bower package name. if it's an origami component,