Fix ingesttopics payload (#88)

Optable · Feb 16, 2024 · 4467645 · 4467645
1 parent 3fdef56
commit 4467645
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/lib/addons/topics-api.test.js b/lib/addons/topics-api.test.js
@@ -23,7 +23,7 @@ describe("OptableSDK - ingestTopics", () => {
 
     // Expectations
     expect(SDK.getTopics).toHaveBeenCalledTimes(1);
-    const expectedTopicsApi = topics.map(topic => JSON.stringify(topic)).join('|');
+    const expectedTopicsApi = topics.map(topic => `taxonomy version ${topic.taxonomyVersion}, topic ${topic.topic}`).join('|');
     expect(SDK.profile).toHaveBeenCalledWith({ topics_api: expectedTopicsApi });
   });
 

diff --git a/lib/addons/topics-api.ts b/lib/addons/topics-api.ts
@@ -24,7 +24,7 @@ OptableSDK.prototype.getTopics = async function(): Promise<BrowsingTopic[]> {
   }
   const getTopicsURL = new URL(siteConfig.getTopicsURL);
   const topicsFrame = document.createElement("iframe");
-  topicsFrame.src = getTopicsURL.toString()
+  topicsFrame.src = getTopicsURL.toString();
   topicsFrame.allow = "browsing-topics " + getTopicsURL.origin;
   topicsFrame.style.display = "none";
 
@@ -35,13 +35,13 @@ OptableSDK.prototype.getTopics = async function(): Promise<BrowsingTopic[]> {
       }
 
       if (event.data.error) {
-        reject(new Error(event.data.error.toString()))
+        reject(new Error(event.data.error.toString()));
         return
       }
 
-      resolve(event.data.result)
-    })
-  })
+      resolve(event.data.result);
+    });
+  });
 
   document.body.appendChild(topicsFrame);
   return topicsPromise;
@@ -54,8 +54,8 @@ OptableSDK.prototype.ingestTopics = function() {
   this.getTopics()
     .then((topics) => {
       if (!topics.length) { return }
-      const topics_api = topics.map(topic => JSON.stringify(topic)).join('|')
-      this.profile({ topics_api })
+      const topics_api = topics.map(topic => `taxonomy version ${topic.taxonomyVersion}, topic ${topic.topic}`).join('|');
+      this.profile({ topics_api });
     })
-    .catch(() => { })
+    .catch(() => { });
 }