Add missing PropTypes

Author: sebprunier

step-3/.eslintrc

@@ -13,6 +13,7 @@
     "modules": true
   },
   "rules": {
+    "jsx-quotes": 1,
     "react/display-name": 0,
     "react/forbid-prop-types": 1,
     "react/jsx-boolean-value": 1,
@@ -27,7 +28,6 @@
     "react/jsx-no-literals": 0,
     "react/jsx-no-undef": 1,
     "react/jsx-pascal-case": 1,
-    "react/jsx-quotes": 1,
     "react/jsx-sort-prop-types": 1,
     "react/jsx-sort-props": 0,
     "react/jsx-uses-react": 1,

step-3/src/components/regions.js

@@ -1,4 +1,4 @@
-import React from 'react'
+import React, { PropTypes } from 'react';
 
 const computeRegionStyle = function(region, selected) {
   let style = {
@@ -12,6 +12,12 @@ const computeRegionStyle = function(region, selected) {
 }
 
 const Regions = React.createClass({
+  propTypes: {
+    onRegionChange: PropTypes.func,
+    regions: PropTypes.arrayOf(PropTypes.string),
+    selected: PropTypes.string
+  },
+
   handleRegionClick(event) {
     this.props.onRegionChange(event.target.textContent);
   },
@@ -23,7 +29,8 @@ const Regions = React.createClass({
           this.props.regions.map(region =>
             <div key={region}
                 style={computeRegionStyle(region, this.props.selected)}
-                onClick={this.handleRegionClick}>
+                onClick={this.handleRegionClick}
+            >
               {region}
             </div>
           )

step-3/src/components/wine-app.js

@@ -27,7 +27,7 @@ const WineApp = React.createClass({
         this.loadWinesByRegion(data[0]);
       })
       .catch(response => {
-        console.log(response);
+        console.error(response); // eslint-disable-line
       });
   },
 
@@ -41,7 +41,7 @@ const WineApp = React.createClass({
         });
       })
       .catch(response => {
-        console.log(response);
+        console.error(response); // eslint-disable-line
       });
   },
 

step-3/src/components/wine-list.js

@@ -1,4 +1,4 @@
-import React from 'react'
+import React, { PropTypes } from 'react';
 
 const computeWineStyle = function(region, selected) {
   let style = {
@@ -12,6 +12,12 @@ const computeWineStyle = function(region, selected) {
 }
 
 const WineList = React.createClass({
+  propTypes: {
+    onWineChange: PropTypes.func,
+    selected: PropTypes.string,
+    wines: PropTypes.arrayOf(PropTypes.object)
+  },
+
   handleWineClick(event) {
     let wineIndex = event.target.getAttribute('data-wineindex');
     this.props.onWineChange(this.props.wines[wineIndex]);
@@ -28,7 +34,8 @@ const WineList = React.createClass({
             <div key={wine.id}
                 data-wineindex={index}
                 style={computeWineStyle(wine, this.props.selected)}
-                onClick={this.handleWineClick}>
+                onClick={this.handleWineClick}
+            >
                 {wine.name}
             </div>
           )

step-3/src/components/wine.js

@@ -48,7 +48,9 @@ const Wine = React.createClass({
     }
     return (
       <div style={Styles.Card}>
-          <img style={Styles.Image} src={`http://localhost:3000/api/wines/${wine.id}/image`} />
+          <img style={Styles.Image}
+              src={`http://localhost:3000/api/wines/${wine.id}/image`}
+          />
           <div style={Styles.Title}>{wine.name}</div>
           <div style={Styles.Info}>
             <span style={Styles.Label}>Type</span>{wine.type}
@@ -65,6 +67,6 @@ const Wine = React.createClass({
       </div>
     )
   }
-})
+});
 
 export default Wine;