Merge pull request stripe#5 from stripe/fix-ssr

dweedon-stripe · web-flow · commit 4775a2110102 · 2020-01-21T12:08:41.000-06:00
Fix ssr
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@stripe/stripe-js",
-  "version": "1.0.0-beta.1",
+  "version": "1.0.0-beta.2",
   "description": "Stripe.js loading utility",
   "main": "dist/stripe.js",
   "module": "dist/stripe.esm.js",
diff --git a/src/index.js b/src/index.js
@@ -51,7 +51,9 @@ const stripePromise = Promise.resolve().then(() => {
 let hasCalledLoadStripe = false;
 export const loadStripe = (...args) => {
   hasCalledLoadStripe = true;
-  return stripePromise.then((Stripe) => Stripe(...args));
+  return stripePromise.then((maybeStripe) =>
+    maybeStripe ? maybeStripe(...args) : null
+  );
 };
 
 const STRIPE_NOT_LOADED_ERROR_TEXT = `Stripe.js has not yet loaded. Instead of calling \`Stripe\` directly, try using the \`loadStripe\` utility from this package.
@@ -74,7 +76,7 @@ const hasUserIncludedScript = () =>
   document.querySelector(`script[src="${V3_URL}"]`) && !hasInjectedScript;
 
 export const Stripe = (...args) => {
-  if (window.Stripe) {
+  if (window && window.Stripe) {
     return window.Stripe(...args);
   }
 

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@stripe/stripe-js",`
`3`		`- "version": "1.0.0-beta.1",`
	`3`	`+ "version": "1.0.0-beta.2",`
`4`	`4`	`"description": "Stripe.js loading utility",`
`5`	`5`	`"main": "dist/stripe.js",`
`6`	`6`	`"module": "dist/stripe.esm.js",`