fix(TDP-1777): Make analytics article_flag data lower case (newsuk#2916)

* Make analytics article_flag data lower case * Sync up styleguide versions * Fix linting
lili2311 · Apr 19, 2022 · 5c4f49a · 5c4f49a
1 parent a382c72
commit 5c4f49a
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/packages/article-skeleton/src/tracking/article-tracking-context.js b/packages/article-skeleton/src/tracking/article-tracking-context.js
@@ -50,7 +50,9 @@ export default Component =>
         template: get(data, "template", "Default"),
         registrationType: getRegistrationType(),
         shared: getSharedStatus(),
-        article_flag: getIsLiveOrBreakingFlag(flags) || "no flag",
+        article_flag: getIsLiveOrBreakingFlag(flags)
+          ? getIsLiveOrBreakingFlag(flags).toLowerCase()
+          : "no flag",
         article_template_name: getIsLiveOrBreakingFlag(flags)
           ? "live template"
           : "standard template"

diff --git a/packages/context/package.json b/packages/context/package.json
@@ -34,7 +34,7 @@
   },
   "homepage": "https://github.com/newsuk/times-components#readme",
   "dependencies": {
-    "@times-components/styleguide": "3.38.34"
+    "@times-components/styleguide": "3.38.35"
   },
   "devDependencies": {
     "@babel/core": "7.4.4",

diff --git a/packages/key-facts/src/key-facts.js b/packages/key-facts/src/key-facts.js
@@ -14,7 +14,9 @@ const KeyFacts = ({ ast, section, headline, isLiveOrBreaking }) => {
 
   const { children: keyFactsItems } = children[0];
 
-  const articleFlag = isLiveOrBreaking || "no flag";
+  const articleFlag = isLiveOrBreaking
+    ? isLiveOrBreaking.toLowerCase()
+    : "no flag";
 
   return (
     <TrackingContextProvider