@quintype/seo 1.40.13-metadata-404.0 → 1.40.14-gsc-errors.0

package/CHANGELOG.md

@@ -2,6 +2,8 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+### [1.40.13](https://github.com/quintype/quintype-node-seo/compare/v1.40.12...v1.40.13) (2022-10-27)
+
 ### [1.40.12](https://github.com/quintype/quintype-node-seo/compare/v1.40.11...v1.40.12) (2022-10-19)
 
 

package/dist/index.cjs.js

@@ -478,7 +478,7 @@ function imageUrl(publisherConfig, s3Key, width, height) {
   const imageSrc = /^https?.*/.test(publisherConfig["cdn-image"]) ? publisherConfig["cdn-image"] : `https://${publisherConfig["cdn-image"]}`;
   width = width || "";
   height = height || "";
-  return `${imageSrc}/${s3Key}?w=${width}&h=${height}&auto=format%2Ccompress&fit=max`;
+  return `${imageSrc}/${s3Key}?w=${width}&h=${height}&auto=format%2Ccompress&fit=max&enlarge=true`;
 }
 
 function generateCommonData(structuredData = {}, story = {}, publisherConfig = {}, timezone) {
@@ -982,33 +982,6 @@ function buildTagsFromTopic(config, tag, url = {}, data) {
   return topicMetaData;
 }
 
-function buildTagsFromNotfoundPage(config, url = {}, data) {
-  const homeSeoData = config["seo-metadata"].find(page => page["owner-type"] === "home") || {
-    data: { description: "" }
-  };
-  const customSeo = lodash.get(data, ["data", "customSeo"], {});
-  const title = customSeo.title || "404 - Page not found  ";
-  const pageTitle = customSeo["page-title"] || title;
-  const description = customSeo.description || homeSeoData.data.description || "404 - Page not found";
-  const tagUrl = `${config["sketches-host"]}${url.pathname}`;
-  const canonicalSlug = tag["canonical-slug"] || url.pathname;
-  const canonicalUrl = `${config["sketches-host"]}${canonicalSlug}`;
-  const ogTitle = customSeo.ogTitle || title;
-  const ogDescription = customSeo.ogDescription || description;
-  const topicMetaData = {
-    title: title,
-    "page-title": pageTitle,
-    description: description,
-    keywords: title,
-    canonicalUrl,
-    ogUrl: tagUrl,
-    ogTitle,
-    ogDescription
-  };
-
-  return topicMetaData;
-}
-
 function buildTagsFromAuthor(config, author, url = {}, data) {
   if (lodash.isEmpty(author)) return;
 
@@ -1125,8 +1098,6 @@ function getSeoData(config, pageType, data, url = {}, seoConfig = {}) {
       return buildTagsFromAuthor(config, lodash.get(data, ["data", "author"], {}), url, data) || getSeoData(config, "home-page", data, url);
     case "static-page":
       return buildTagsFromStaticPage(config, lodash.get(data, ["data", "page"], {}), url, data) || getSeoData(config, "home-page", data, url);
-    case "not-found":
-      return buildTagsFromNotfoundPage(config, url, data) || getSeoData(config, "home-page", data, url);
     case "shell":
       return getShellSeoData(config);
     default:
@@ -1321,7 +1292,7 @@ class SEO {
    */
   constructor(seoConfig = {}) {
     this.seoConfig = seoConfig;
-    this.generators = (seoConfig.generators || [TextTags, ImageTags, AuthorTags, StaticTags, StructuredDataTags, StoryAmpTags]).concat(seoConfig.extraGenerators || []);
+    this.generators = (seoConfig.generators || [TextTags, ImageTags, AuthorTags, StaticTags, StructuredDataTags]).concat(seoConfig.extraGenerators || []);
   }
 
   getMetaTags(config, pageType, data, params = {}) {

package/index.js

@@ -1,23 +1,35 @@
 import { flatMap, get, omit, uniqBy } from "lodash";
 import React from "react";
 import ReactDomServer from "react-dom/server";
-import { StoryAmpTags } from './src/amp-tags.js';
-import { AuthorTags } from './src/author-tags.js';
-import { generateStaticData, generateStructuredData } from './src/generate-common-seo';
-import { ImageTags } from './src/image-tags.js';
-import { StaticTags } from './src/static-tags.js';
-import { StructuredDataTags } from './src/structured-data/structured-data-tags.js';
-import { getTitle, TextTags } from './src/text-tags.js';
+import { StoryAmpTags } from "./src/amp-tags.js";
+import { AuthorTags } from "./src/author-tags.js";
+import { generateStaticData, generateStructuredData } from "./src/generate-common-seo";
+import { ImageTags } from "./src/image-tags.js";
+import { StaticTags } from "./src/static-tags.js";
+import { StructuredDataTags } from "./src/structured-data/structured-data-tags.js";
+import { getTitle, TextTags } from "./src/text-tags.js";
 
-
-export { TextTags, StaticTags, AuthorTags, ImageTags, StructuredDataTags, StoryAmpTags, generateStaticData, generateStructuredData };
+export {
+  TextTags,
+  StaticTags,
+  AuthorTags,
+  ImageTags,
+  StructuredDataTags,
+  StoryAmpTags,
+  generateStaticData,
+  generateStructuredData,
+};
 
 function tagToKey(tag) {
   switch (tag.tag || "meta") {
-    case "meta": return `meta-${tag.name || tag.itemprop || "name"}-${tag.property || "property"}`;
-    case "link": return `link-${tag.rel}`;
-    case "title": return `title`;
-    default: return Math.random().toString();
+    case "meta":
+      return `meta-${tag.name || tag.itemprop || "name"}-${tag.property || "property"}`;
+    case "link":
+      return `link-${tag.rel}`;
+    case "title":
+      return `title`;
+    default:
+      return Math.random().toString();
   }
 }
 
@@ -28,7 +40,9 @@ export class MetaTagList {
 
   toString() {
     const uniqueTags = uniqBy(this.tags.reverse(), tagToKey).reverse();
-    return ReactDomServer.renderToStaticMarkup(uniqueTags.map(tag => React.createElement(tag.tag || "meta", omit(tag, "tag"))));
+    return ReactDomServer.renderToStaticMarkup(
+      uniqueTags.map((tag) => React.createElement(tag.tag || "meta", omit(tag, "tag")))
+    );
   }
 
   addTag() {
@@ -93,12 +107,16 @@ export class SEO {
    */
   constructor(seoConfig = {}) {
     this.seoConfig = seoConfig;
-    this.generators = (seoConfig.generators || [TextTags, ImageTags, AuthorTags, StaticTags, StructuredDataTags, StoryAmpTags]).concat(seoConfig.extraGenerators || []);
+    this.generators = (
+      seoConfig.generators || [TextTags, ImageTags, AuthorTags, StaticTags, StructuredDataTags]
+    ).concat(seoConfig.extraGenerators || []);
   }
 
   getMetaTags(config, pageType, data, params = {}) {
     pageType = get(this.seoConfig, ["pageTypeAliases", pageType], pageType);
-    return new MetaTagList(flatMap(this.generators, generator => generator(this.seoConfig, config, pageType, data, params)));
+    return new MetaTagList(
+      flatMap(this.generators, (generator) => generator(this.seoConfig, config, pageType, data, params))
+    );
   }
 
   getTitle(config, pageType, data, params = {}) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quintype/seo",
-  "version": "1.40.13-metadata-404.0",
+  "version": "1.40.14-gsc-errors.0",
   "description": "SEO Modules for Quintype",
   "main": "dist/index.cjs.js",
   "repository": "https://github.com/quintype/quintype-node-seo",

package/src/structured-data/structured-data-tags.js

@@ -34,7 +34,7 @@ function imageUrl(publisherConfig, s3Key, width, height) {
     : `https://${publisherConfig["cdn-image"]}`;
   width = width || "";
   height = height || "";
-  return `${imageSrc}/${s3Key}?w=${width}&h=${height}&auto=format%2Ccompress&fit=max`;
+  return `${imageSrc}/${s3Key}?w=${width}&h=${height}&auto=format%2Ccompress&fit=max&enlarge=true`;
 }
 
 function generateCommonData(structuredData = {}, story = {}, publisherConfig = {}, timezone) {

package/src/text-tags.js

@@ -81,33 +81,6 @@ function buildTagsFromTopic(config, tag, url = {}, data) {
   return topicMetaData;
 }
 
-function buildTagsFromNotfoundPage(config, url = {}, data) {
-  const homeSeoData = config["seo-metadata"].find((page) => page["owner-type"] === "home") || {
-    data: { description: "" },
-  };
-  const customSeo = get(data, ["data", "customSeo"], {});
-  const title = customSeo.title || "404 - Page not found  ";
-  const pageTitle = customSeo["page-title"] || title;
-  const description = customSeo.description || homeSeoData.data.description || "404 - Page not found";
-  const tagUrl = `${config["sketches-host"]}${url.pathname}`;
-  const canonicalSlug = tag["canonical-slug"] || url.pathname;
-  const canonicalUrl = `${config["sketches-host"]}${canonicalSlug}`;
-  const ogTitle = customSeo.ogTitle || title;
-  const ogDescription = customSeo.ogDescription || description;
-  const topicMetaData = {
-    title: title,
-    "page-title": pageTitle,
-    description: description,
-    keywords: title,
-    canonicalUrl,
-    ogUrl: tagUrl,
-    ogTitle,
-    ogDescription,
-  };
-
-  return topicMetaData;
-}
-
 function buildTagsFromAuthor(config, author, url = {}, data) {
   if (isEmpty(author)) return;
 
@@ -247,8 +220,6 @@ function getSeoData(config, pageType, data, url = {}, seoConfig = {}) {
         buildTagsFromStaticPage(config, get(data, ["data", "page"], {}), url, data) ||
         getSeoData(config, "home-page", data, url)
       );
-    case "not-found":
-      return buildTagsFromNotfoundPage(config, url, data) || getSeoData(config, "home-page", data, url);
     case "shell":
       return getShellSeoData(config);
     default: