PR feedback

Dockerfile

@@ -4,6 +4,8 @@ ENV NODE_ENV production
 ENV API_URL http://api.quran.com:3000
 ENV SENTRY_KEY_CLIENT https://44c105328ae544ae9928f9eb74b40061@app.getsentry.com/80639
 ENV SENTRY_KEY_SERVER https://44c105328ae544ae9928f9eb74b40061:41ca814d33124e04ab450104c3938cb1@app.getsentry.com/80639
+# It's okay because it's only the APP ID
+ENV FACEBOOK_APP_ID 1557596491207315
 ENV PORT 8000
 ENV NODE_PATH "./src"
 

src/components/IndexHeader/Nav/index.js

@@ -2,7 +2,6 @@ import React, { Component, PropTypes } from 'react';
 import Link from 'react-router/lib/Link';
 import { connect } from 'react-redux';
 
-
 export class IndexHeaderNav extends Component {
   static propTypes = {
     user: PropTypes.object
@@ -12,15 +11,15 @@ export class IndexHeaderNav extends Component {
     open: false
   };
 
-  openNav(e) {
-    e.preventDefault();
+  openNav(event) {
+    event.preventDefault();
 
     this.setState({open: !this.state.open});
   }
 
   links() {
     const { user } = this.props;
-    let classNames = `links ${this.state.open ? 'open' : ''}`;
+    const classNames = `links ${this.state.open ? 'open' : ''}`;
 
     return (
       <ul className={classNames}>

src/containers/Login/index.js

@@ -1,16 +1,9 @@
-import React, { Component } from 'react';
+import React from 'react';
 
 import FacebookButton from 'components/FacebookButton';
 
-export default class Login extends Component {
-  something() {
-
-  }
-  render() {
-    return (
-      <div>
-        <FacebookButton />
-      </div>
-    );
-  }
-}
+export default () => (
+  <div>
+    <FacebookButton />
+  </div>
+);

src/containers/Profile/index.js

@@ -7,6 +7,7 @@ import Col from 'react-bootstrap/lib/Col';
 import Image from 'react-bootstrap/lib/Image';
 
 import QuranNav from 'components/QuranNav';
+import userType from 'types/userType';
 
 const styles = require('./style.scss');
 
@@ -17,10 +18,9 @@ const styles = require('./style.scss');
 )
 export default class Profile extends Component {
   static propTypes = {
-    user: PropTypes.object
+    user: PropTypes.shape(userType)
   };
 
-  something() {}
   render() {
     const { user } = this.props;
 

src/types/userType.js

@@ -0,0 +1,13 @@
+import { PropTypes } from 'react';
+
+export default {
+  provider: PropTypes.string,
+  uid: PropTypes.string,
+  firstName: PropTypes.string,
+  lastName: PropTypes.string,
+  username: PropTypes.string,
+  link: PropTypes.string,
+  image: PropTypes.string,
+  email: PropTypes.string,
+  name: PropTypes.string
+};