Merge pull request #465 from paypal/DTPPCPSDK-1482

Set data-js-sdk-library default value
paypal · Mar 19, 2024 · 199e978 · 199e978
2 parents f829b81 + 9d90088
commit 199e978
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/.changeset/brown-elephants-behave.md b/.changeset/brown-elephants-behave.md
@@ -0,0 +1,5 @@
+---
+"@paypal/paypal-js": patch
+---
+
+Set default data-js-sdk-library value
diff --git a/packages/paypal-js/src/load-script.test.ts b/packages/paypal-js/src/load-script.test.ts
@@ -46,7 +46,7 @@ describe("loadScript()", () => {
 
         // simulate the script already being loaded
         document.head.innerHTML =
-            '<script src="https://www.paypal.com/sdk/js?client-id=test"></script>';
+            '<script src="https://www.paypal.com/sdk/js?client-id=test" data-js-sdk-library="paypal-js"></script>';
         vi.stubGlobal("paypal", { version: "5" });
 
         const response = await loadScript({ clientId: "test" });

diff --git a/packages/paypal-js/src/load-script.ts b/packages/paypal-js/src/load-script.ts
@@ -22,6 +22,10 @@ export function loadScript(
     const namespace = attributes["data-namespace"] || "paypal";
     const existingWindowNamespace = getPayPalWindowNamespace(namespace);
 
+    if (!attributes["data-js-sdk-library"]) {
+        attributes["data-js-sdk-library"] = "paypal-js";
+    }
+
     // resolve with the existing global paypal namespace when a script with the same params already exists
     if (findScript(url, attributes) && existingWindowNamespace) {
         return PromisePonyfill.resolve(existingWindowNamespace);