Fix linting errors

Rizchel Dayao · Rizchel Dayao · commit b0f7a70149c3 · 2018-03-06T14:08:49.000-07:00
diff --git a/public/index.html b/public/index.html
@@ -4,9 +4,9 @@
     <meta charset="utf-8">
     <meta name="viewport" content="width=device-width, initial-scale=1">
     <link rel="shortcut icon" href="">
-    <title>React Redux Example</title>
+    <title>Movie Finder</title>
   </head>
   <body>
-    <div id="root"></div>
+    <main id="root"></main>
   </body>
 </html>
diff --git a/src/actions/index.js b/src/actions/index.js
@@ -1,30 +1,31 @@
-import fetch from 'isomorphic-fetch';
-import ActionTypes from '../constants';
+import fetch from 'isomorphic-fetch'
+import ActionTypes from '../constants'
 
-function retrieveMoviesStart() {
+// Actions for retrieving movie
+const retrieveMoviesStart = () => {
   return {
     type: ActionTypes.RETRIEVE_MOVIES_START,
   };
 }
 
-function retrieveMoviesError(error) {
+const retrieveMoviesError = (error) => {
   return {
     type: ActionTypes.RETRIEVE_MOVIES_ERROR,
     payload: error
   };
 }
 
-function retrieveMoviesSuccess(response) {
+const retrieveMoviesSuccess = (response) => {
   return {
     type: ActionTypes.RETRIEVE_MOVIES_SUCCESS,
     payload: response
   };
 }
 
-export function retrieveMovies(userInput) {
+export const retrieveMovies = (userInput) => {
   return dispatch => {
   // Insert your api key on line 30
-  const URL = `http://www.omdbapi.com/?apikey=d8c6a83b&t=${userInput}`;
+  const URL = `http://www.omdbapi.com/?apikey=[apiKey]&t=${userInput}`;
   let request = new Request(URL, {
     method: 'GET',
   });
diff --git a/src/components/Input.js b/src/components/Input.js
@@ -9,6 +9,7 @@ class Input extends Component {
     this.handleResponse = this.handleResponse.bind(this);
   }
 
+  //Callbacks for when user input changes or 'search' is clicked
   handleChange(e) {
     const { handleChange } = this.props;
     handleChange(e);
@@ -35,7 +36,7 @@ class Input extends Component {
   }
 }
 
-Input.propTypes = {
+Input.PropTypes = {
   input: PropTypes.string,
   handleChange: PropTypes.func,
   handleKeyPress: PropTypes.func,
diff --git a/src/components/ListItem.js b/src/components/ListItem.js
@@ -1,6 +1,4 @@
-import React, { PropTypes, Component } from 'react';
-import { connect } from 'react-redux';
-import '../style/main.css';
+import React, { PropTypes, Component } from 'react'
 
 class ListItem extends Component {
   render() {
diff --git a/src/containers/App.js b/src/containers/App.js
@@ -1,10 +1,6 @@
-import React, { Component, PropTypes } from 'react'
-import { connect } from 'react-redux'
+import React, { Component } from 'react'
 import List from './List'
-import InputSet from './InputSet';
-import * as MoviesAction from '../actions'
-import { bindActionCreators } from 'redux'
-import '../style/main.css';
+import InputSet from './InputSet'
 
 class App extends Component {
 
diff --git a/src/containers/InputSet.js b/src/containers/InputSet.js
@@ -3,7 +3,6 @@ import { connect } from 'react-redux'
 import Input from '../components/Input'
 import * as MoviesAction from '../actions'
 import { bindActionCreators } from 'redux'
-import '../style/main.css';
 
 class InputSet extends Component {
   constructor(props) {
@@ -17,15 +16,19 @@ class InputSet extends Component {
     this.handleResponse = this.handleResponse.bind(this)
   }
 
+  // Function for when 'search' is clicked and retrieves movies
   handleResponse(input) {
     const { retrieveMovies } = this.props;
+    // eslint-disable-next-line
     const result = retrieveMovies(input);
   }
 
+  // Function for when text in input changes
   handleChange(e) {
     this.setState({ input: e.target.value });
   }
 
+  // Function for 'enter' key press
   handleKeyPress(target) {
     const { input } = this.state;
    if(target.charCode===13){
@@ -35,7 +38,6 @@ class InputSet extends Component {
 
   render() {
     const { input } = this.state;
-    const { handleResponse } = this.props;
     return (
       <div className="input-set">
         <Input movieValue={input} handleChange={this.handleChange} handleKeyPress={this.handleKeyPress} handleResponse={this.handleResponse}/>
@@ -44,17 +46,20 @@ class InputSet extends Component {
   }
 }
 
+// Set state and action as proptype
 InputSet.propTypes = {
   movies: PropTypes.object,
   retrieveMovies: PropTypes.func
 }
 
+// Map Redux state to props
 function mapStateToProps(state) {
   return {
     movies: state.movies
   };
 }
 
+// Map action to props
 function mapDispatchToProps(dispatch) {
   return bindActionCreators(Object.assign({}, MoviesAction), dispatch);
 }
diff --git a/src/containers/List.js b/src/containers/List.js
@@ -1,8 +1,7 @@
-import React, { PropTypes, Component } from 'react';
-import { connect } from 'react-redux';
-import '../style/main.css';
-import ListItem from '../components/ListItem';
-import createList from '../utils';
+import React, { Component } from 'react'
+import { connect } from 'react-redux'
+import ListItem from '../components/ListItem'
+import createList from '../utils'
 
 class List extends Component {
   render() {
@@ -16,6 +15,7 @@ class List extends Component {
   }
 }
 
+// Map Redux state to props
 function mapStateToProps(state) {
   return {
     movies: state.movies
diff --git a/src/reducers/index.js b/src/reducers/index.js
@@ -1,5 +1,5 @@
-import { combineReducers } from 'redux';
-import { routerStateReducer } from 'redux-router';
+import { combineReducers } from 'redux'
+import { routerStateReducer } from 'redux-router'
 import ActionTypes from '../constants'
 
 const initialState = {
diff --git a/src/utils/index.js b/src/utils/index.js
@@ -1,4 +1,4 @@
-import React from 'react';
+import React from 'react'
 
 export default function createList(movies) {
   const myList = [];

Original file line number	Diff line number	Diff line change
`@@ -9,6 +9,7 @@ class Input extends Component {`
`9`	`9`	`this.handleResponse = this.handleResponse.bind(this);`
`10`	`10`	`}`
`11`	`11`
	`12`	`+ //Callbacks for when user input changes or 'search' is clicked`
`12`	`13`	`handleChange(e) {`
`13`	`14`	`const { handleChange } = this.props;`
`14`	`15`	`handleChange(e);`
`@@ -35,7 +36,7 @@ class Input extends Component {`
`35`	`36`	`}`
`36`	`37`	`}`
`37`	`38`
`38`		`-Input.propTypes = {`
	`39`	`+Input.PropTypes = {`
`39`	`40`	`input: PropTypes.string,`
`40`	`41`	`handleChange: PropTypes.func,`
`41`	`42`	`handleKeyPress: PropTypes.func,`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-import React from 'react';`
	`1`	`+import React from 'react'`
`2`	`2`
`3`	`3`	`export default function createList(movies) {`
`4`	`4`	`const myList = [];`