Merge pull request #326 from reactioncommerce/fix-325-aldeed-stripe-f…

…orm-error

Fix StripeForm error

package/README.md

@@ -3,16 +3,18 @@
 ## Getting Started
 
 ```bash
-npm install --save react@16.4.1 prop-types@15.6.2 styled-components@3.3.3 @reactioncommerce/components
+npm install --save react@16.4.2 prop-types@15.6.2 styled-components@3.3.3 reacto-form@0.0.2 @reactioncommerce/components-context@1.0.0 @reactioncommerce/components
 ```
 
 or
 
 ```bash
-yarn add react@16.4.1 prop-types@15.6.2 styled-components@3.3.3 @reactioncommerce/components
+yarn add react@16.4.2 prop-types@15.6.2 styled-components@3.3.3 reacto-form@0.0.2 @reactioncommerce/components-context@1.0.0 @reactioncommerce/components
 ```
 
-Note that the minimum required React version is 16.4.1 because this package uses newer APIs like `createContext` and `forwardRef`. The `react`, `prop-types`, and `styled-components` packages are peer dependencies, which means that you must install the proper versions in your app. They are not included with this package.
+Note that the minimum required React version is 16.4.1 because this package uses newer APIs like `createContext` and `forwardRef`. The `react`, `prop-types`, `@reactioncommerce/components-context`, `reacto-form`, and `styled-components` packages are peer dependencies, which means that you must install the proper versions in your app. They are not included with this package.
+
+> If you use the `StripeForm` component, then you must also install `react-stripe-elements@2.0.1`. It is an optional peer dependency.
 
 Then import a component:
 

package/src/components/StripeForm/v1/StripeForm.js

@@ -7,7 +7,7 @@ import {
   PostalCodeElement,
   injectStripe
 } from "react-stripe-elements";
-import styled, { withTheme } from "styled-components";
+import styled from "styled-components";
 import { withComponents } from "@reactioncommerce/components-context";
 import { applyTheme, withStripeElements } from "../../../utils";
 
@@ -243,4 +243,8 @@ class StripeForm extends Component {
   }
 }
 
-export default withTheme(withComponents(withStripeElements(injectStripe(StripeForm))));
+// We should be using `withTheme` here, but it seems to cause serious errors in the
+// deployed app, and does not work anyway. Need to investigate why `withComponents`
+// works and does not cause errors but `withTheme` does not work. It is surely
+// something related to the iframes that Stripe uses.
+export default withComponents(withStripeElements(injectStripe(StripeForm)));

styleguide/src/sections/InstallingandImporting.md

@@ -1,18 +1,18 @@
 #### Install
 
-With NPM:
-
 ```bash
-npm install --save react@16.4.1 prop-types@15.6.2 styled-components@3.3.3 @reactioncommerce/components
+npm install --save react@16.4.2 prop-types@15.6.2 styled-components@3.3.3 reacto-form@0.0.2 @reactioncommerce/components-context@1.0.0 @reactioncommerce/components
 ```
 
-With Yarn:
+or
 
 ```bash
-yarn add react@16.4.1 prop-types@15.6.2 styled-components@3.3.3 @reactioncommerce/components
+yarn add react@16.4.2 prop-types@15.6.2 styled-components@3.3.3 reacto-form@0.0.2 @reactioncommerce/components-context@1.0.0 @reactioncommerce/components
 ```
 
-Note that the minimum required React version is 16.4.1 because this package uses newer APIs like `createContext` and `forwardRef`. The `react`, `prop-types`, and `styled-components` packages are peer dependencies, which means that you must install the proper versions in your app. They are not included with this package.
+Note that the minimum required React version is 16.4.1 because this package uses newer APIs like `createContext` and `forwardRef`. The `react`, `prop-types`, `@reactioncommerce/components-context`, `reacto-form`, and `styled-components` packages are peer dependencies, which means that you must install the proper versions in your app. They are not included with this package.
+
+> If you use the `StripeForm` component, then you must also install `react-stripe-elements@2.0.1`. It is an optional peer dependency.
 
 ##### Verify Peer Dependencies