Fix FE when tag name is undefined (#2146)

Partly resolves #2106
kedro-org · Oct 25, 2024 · ddb2386 · ddb2386
1 parent 9996c99
commit ddb2386
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/selectors/tags.js b/src/selectors/tags.js
@@ -1,5 +1,6 @@
 import { createSelector } from 'reselect';
 import { getPipelineTagIDs } from './pipeline';
+import { prettifyName } from '../utils';
 
 const getNodeTags = (state) => state.node.tags;
 const getTagName = (state) => state.tag.name;
@@ -14,7 +15,7 @@ export const getTagData = createSelector(
   (tagIDs, tagName, tagActive, tagEnabled) =>
     tagIDs.sort().map((id) => ({
       id,
-      name: tagName[id],
+      name: tagName[id] || prettifyName(id),
       active: Boolean(tagActive[id]),
       enabled: Boolean(tagEnabled[id]),
     }))

diff --git a/src/utils/index.js b/src/utils/index.js
@@ -121,6 +121,9 @@ export const stripNamespace = (str) => {
  * @returns {String} The string with or without replaced values
  */
 export const prettifyName = (str) => {
+  if (!str) {
+    return '';
+  }
   const replacedString = str
     .replace(/-/g, ' ')
     .replace(/_/g, ' ')