@raketa-cloud/admin 1.10.3 → 1.10.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@raketa-cloud/admin",
-  "version": "1.10.3",
+  "version": "1.10.4",
   "main": "src/index.js",
   "files": [
     "src/",

package/src/helpers/content_tasks_helper.js

@@ -24,26 +24,21 @@ export const buildResourceUrl = (resource, globalConfig) => {
   return `/${locale}/${route}/${id}`;
 };
 
-export const hasUnattachedKeys = (task) => {
-  if (!task) return false;
+export const unattachedKeys = (task) => {
+  const keys = task?.lokaliseWorkflow?.settings?.unattached_keys;
+  const values = keys?.names || keys?.ids;
 
-  const unattachedKeys = task.lokaliseWorkflow?.settings?.unattached_keys;
-  const ids = unattachedKeys?.ids;
+  if (Array.isArray(values)) return values;
 
-  if (Array.isArray(ids)) {
-    return ids.length > 0;
-  }
-
-  return Boolean(ids);
+  return values ? [values] : [];
 };
 
-export const unattachedKeyIds = (task) => {
-  const unattachedKeys = task?.lokaliseWorkflow?.settings?.unattached_keys;
-  const ids = unattachedKeys?.ids;
+export const hasUnattachedKeys = (task) => unattachedKeys(task).length > 0;
 
-  if (Array.isArray(ids)) {
-    return ids.join(", ");
-  }
+export const unattachedKeyUrl = (task, key) => {
+  const keys = task?.lokaliseWorkflow?.settings?.unattached_keys;
+
+  if (!keys?.names || !keys.url) return null;
 
-  return ids ? String(ids) : "";
+  return `${keys.url}${encodeURIComponent(key)}`;
 };

package/src/tests/helpers/content_tasks_helper.test.js

@@ -2,7 +2,8 @@ import {
   workflowsMap,
   buildResourceUrl,
   hasUnattachedKeys,
-  unattachedKeyIds,
+  unattachedKeys,
+  unattachedKeyUrl,
 } from "../../helpers/content_tasks_helper";
 
 describe("content_tasks_helper", () => {
@@ -114,46 +115,89 @@ describe("content_tasks_helper", () => {
       ).toBe(false);
     });
 
-    test("returns true when task is in progress and has unattached key ids", () => {
+    test("returns true when the task has unattached key ids or names", () => {
       expect(
         hasUnattachedKeys({
-          completedAt: null,
-          failedAt: null,
           lokaliseWorkflow: {
             settings: { unattached_keys: { ids: [101, 102] } },
           },
         }),
       ).toBe(true);
+      expect(
+        hasUnattachedKeys({
+          lokaliseWorkflow: {
+            settings: { unattached_keys: { names: ["key_a"] } },
+          },
+        }),
+      ).toBe(true);
     });
   });
 
-  describe("unattachedKeyIds", () => {
-    test("returns empty string when no unattached keys exist", () => {
-      expect(unattachedKeyIds(null)).toBe("");
-      expect(unattachedKeyIds({})).toBe("");
-      expect(
-        unattachedKeyIds({
-          lokaliseWorkflow: { settings: { unattached_keys: {} } },
-        }),
-      ).toBe("");
+  describe("unattachedKeys", () => {
+    test("returns an empty array when no unattached keys exist", () => {
+      expect(unattachedKeys(null)).toEqual([]);
+      expect(unattachedKeys({})).toEqual([]);
     });
 
-    test("returns comma separated string of ids", () => {
+    test("prefers names and falls back to ids", () => {
       expect(
-        unattachedKeyIds({
+        unattachedKeys({
           lokaliseWorkflow: {
-            settings: { unattached_keys: { ids: [101, 102, 103] } },
+            settings: {
+              unattached_keys: { names: ["key_a", "key_b"], ids: [101, 102] },
+            },
           },
         }),
-      ).toBe("101, 102, 103");
-
+      ).toEqual(["key_a", "key_b"]);
       expect(
-        unattachedKeyIds({
+        unattachedKeys({
           lokaliseWorkflow: {
-            settings: { unattached_keys: { ids: ["key_a", "key_b"] } },
+            settings: { unattached_keys: { ids: [101, 102] } },
           },
         }),
-      ).toBe("key_a, key_b");
+      ).toEqual([101, 102]);
+    });
+  });
+
+  describe("unattachedKeyUrl", () => {
+    const task = {
+      lokaliseWorkflow: {
+        settings: {
+          unattached_keys: {
+            names: ["key one"],
+            url: "https://app.lokalise.com/project/123/?search=",
+          },
+        },
+      },
+    };
+
+    test("builds a Lokalise URL for a key name", () => {
+      expect(unattachedKeyUrl(task, "key one")).toBe(
+        "https://app.lokalise.com/project/123/?search=key%20one",
+      );
+    });
+
+    test("returns null for legacy ids or when the URL is missing", () => {
+      expect(
+        unattachedKeyUrl(
+          {
+            lokaliseWorkflow: {
+              settings: { unattached_keys: { ids: [101] } },
+            },
+          },
+          101,
+        ),
+      ).toBeNull();
+      expect(
+        unattachedKeyUrl(
+          {
+            lokaliseWorkflow: {
+              settings: { unattached_keys: { names: ["key_a"] } },
+            },
+          },
+          "key_a",
+        ),
+      ).toBeNull();
     });
   });
 });

package/src/views/_shared/_content_tasks.ejs

@@ -66,7 +66,15 @@
 
             <% if (isMarked) { %>
               <div class="body-3 text-raketa-foreground/50">
-                <%= content_tasks.unattachedKeyIds(task) %>
+                <% content_tasks.unattachedKeys(task).forEach((key, index) => { %>
+                  <% if (index > 0) { %>, <% } %>
+                  <% const keyUrl = content_tasks.unattachedKeyUrl(task, key) %>
+                  <% if (keyUrl) { %>
+                    <a href="<%= keyUrl %>" target="_blank" class="btn-text"><%= key %></a>
+                  <% } else { %>
+                    <%= key %>
+                  <% } %>
+                <% }) %>
               </div>
             <% } %>
           </td>