Add onLineItemsChange option to EmbeddedCheckoutProvider (#536)

stripe · Oct 9, 2024 · c22ae09 · c22ae09
1 parent f0158b9
commit c22ae09
Show file tree

Hide file tree

Showing 4 changed files with 48 additions and 5 deletions.
diff --git a/package.json b/package.json
@@ -69,7 +69,7 @@
     "@rollup/plugin-replace": "^5.0.5",
     "@rollup/plugin-terser": "^0.4.4",
     "@storybook/react": "^6.5.0-beta.8",
-    "@stripe/stripe-js": "^4.3.0",
+    "@stripe/stripe-js": "^4.8.0",
     "@testing-library/jest-dom": "^5.16.4",
     "@testing-library/react": "^13.1.1",
     "@testing-library/react-hooks": "^8.0.0",

diff --git a/src/components/EmbeddedCheckoutProvider.test.tsx b/src/components/EmbeddedCheckoutProvider.test.tsx
@@ -414,6 +414,37 @@ describe('EmbeddedCheckoutProvider', () => {
     );
   });
 
+  it('does not allow changes to onLineItemsChange option', async () => {
+    const optionsProp1 = {
+      fetchClientSecret,
+      onLineItemsChange: () => Promise.resolve({type: 'accept' as const}),
+    };
+    const optionsProp2 = {
+      fetchClientSecret,
+      onLineItemsChange: () => Promise.resolve({type: 'reject' as const}),
+    };
+
+    // Silence console output so test output is less noisy
+    consoleWarn.mockImplementation(() => {});
+    const {rerender} = render(
+      <EmbeddedCheckoutProvider
+        stripe={mockStripe}
+        options={optionsProp1}
+      ></EmbeddedCheckoutProvider>
+    );
+    await act(() => mockEmbeddedCheckoutPromise);
+
+    rerender(
+      <EmbeddedCheckoutProvider
+        stripe={mockStripe}
+        options={optionsProp2}
+      ></EmbeddedCheckoutProvider>
+    );
+    expect(consoleWarn).toHaveBeenCalledWith(
+      'Unsupported prop change on EmbeddedCheckoutProvider: You cannot change the onLineItemsChange option after setting it.'
+    );
+  });
+
   it('destroys Embedded Checkout when the component unmounts', async () => {
     const {rerender} = render(
       <div>

diff --git a/src/components/EmbeddedCheckoutProvider.tsx b/src/components/EmbeddedCheckoutProvider.tsx
@@ -57,6 +57,9 @@ interface EmbeddedCheckoutProviderProps {
     onShippingDetailsChange?: (
       event: stripeJs.StripeEmbeddedCheckoutShippingDetailsChangeEvent
     ) => Promise<stripeJs.ResultAction>;
+    onLineItemsChange?: (
+      event: stripeJs.StripeEmbeddedCheckoutLineItemsChangeEvent
+    ) => Promise<stripeJs.ResultAction>;
   };
 }
 
@@ -220,6 +223,15 @@ export const EmbeddedCheckoutProvider: FunctionComponent<PropsWithChildren<
         'Unsupported prop change on EmbeddedCheckoutProvider: You cannot change the onShippingDetailsChange option after setting it.'
       );
     }
+
+    if (
+      prevOptions.onLineItemsChange != null &&
+      options.onLineItemsChange !== prevOptions.onLineItemsChange
+    ) {
+      console.warn(
+        'Unsupported prop change on EmbeddedCheckoutProvider: You cannot change the onLineItemsChange option after setting it.'
+      );
+    }
   }, [prevOptions, options]);
 
   return (

diff --git a/yarn.lock b/yarn.lock
@@ -2350,10 +2350,10 @@
     regenerator-runtime "^0.13.7"
     resolve-from "^5.0.0"
 
-"@stripe/stripe-js@^4.3.0":
-  version "4.3.0"
-  resolved "https://registry.yarnpkg.com/@stripe/stripe-js/-/stripe-js-4.3.0.tgz#5e8fe1e654413698f33cbc7008fef1e86e16b030"
-  integrity sha512-bf8MxzzgD3dybtyIJUQSDMqxjEkJfsmj9IdRqDv609Zw08R41O7eoIy0f8KY41u8MbaFOYsn+XGJZtg1xwR2wQ==
+"@stripe/stripe-js@^4.8.0":
+  version "4.8.0"
+  resolved "https://registry.yarnpkg.com/@stripe/stripe-js/-/stripe-js-4.8.0.tgz#ba78f775347cb30e93e13aa408b345de15e3ff80"
+  integrity sha512-+4Cb0bVHlV4BJXxkJ3cCLSLuWxm3pXKtgcRacox146EuugjCzRRII5T5gUMgL4HpzrBLVwVxjKaZqntNWAXawQ==
 
 "@testing-library/dom@^8.5.0":
   version "8.13.0"