From 7692e158022b84eaab86c2ff83af2314ffbf0cd4 Mon Sep 17 00:00:00 2001
From: LordMathis <matus@namesny.com>
Date: Sat, 23 Nov 2019 22:36:20 +0100
Subject: [PATCH] Fix some eslint issues

---
 src/components/About.js                          |  1 -
 src/components/App.js                            |  2 +-
 src/components/Blog.js                           |  8 ++++----
 src/components/Column.js                         | 10 +++++-----
 src/components/Header.js                         | 10 +++++++---
 src/components/Home.js                           |  2 +-
 src/components/Navbar.js                         |  1 -
 src/components/Resume.js                         |  5 +++--
 src/components/SocialLinks.js                    | 16 ++++++++--------
 src/components/index.js                          |  1 -
 src/containers/ContentContainer.js               |  8 ++++----
 .../NotFoundContainer.js}                        | 10 +++++-----
 src/containers/index.js                          |  1 +
 13 files changed, 39 insertions(+), 36 deletions(-)
 rename src/{components/NotFoundWrapper.js => containers/NotFoundContainer.js} (71%)

diff --git a/src/components/About.js b/src/components/About.js
index 4a2ce40..edb4782 100644
--- a/src/components/About.js
+++ b/src/components/About.js
@@ -3,7 +3,6 @@ import React, { Component } from 'react'
 import { Spinner, Header } from '.'
 import '../stylesheets/globals.scss'
 import contentStyle from '../stylesheets/content.scss'
-import style from './About.scss'
 import MarkdownIt from 'markdown-it'
 
 export default class About extends Component {
diff --git a/src/components/App.js b/src/components/App.js
index 1c49409..ea03911 100644
--- a/src/components/App.js
+++ b/src/components/App.js
@@ -1,4 +1,4 @@
-import { NotFoundWrapper } from '.'
+import { NotFoundWrapper } from '../containers'
 import React, { Component } from 'react'
 import routes from '../utils/routes'
 import { Route, Switch } from 'react-router-dom'
diff --git a/src/components/Blog.js b/src/components/Blog.js
index 8b8d4d0..ca6d940 100644
--- a/src/components/Blog.js
+++ b/src/components/Blog.js
@@ -22,12 +22,12 @@ export default class Blog extends Component {
         </div>
       )
     }
-    
-    let posts = this.props.posts.sort((a, b) => {
+
+    const posts = this.props.posts.sort((a, b) => {
       return new Date(b.published) - new Date(a.published)
     })
-    let postsHTML = posts.map((post) => 
-      <div className={styles.postListItem} role="listitem">
+    const postsHTML = posts.map((post) =>
+      <div key={post.title} className={styles.postListItem} role="listitem">
         <div className={styles.postHeader} >
           <a href={post.link} className={styles.postTitle}>{post.title}</a>
           <span className={styles.postDate}>{post.published}</span>
diff --git a/src/components/Column.js b/src/components/Column.js
index 3333afe..3fbc84f 100644
--- a/src/components/Column.js
+++ b/src/components/Column.js
@@ -5,11 +5,11 @@ import styles from './Column.scss'
 
 export default class Column extends Component {
   static propTypes = {
-      children: PropTypes.oneOfType([
-          PropTypes.arrayOf(PropTypes.node),
-          PropTypes.node
-      ]).isRequired,
-      left: PropTypes.bool
+    children: PropTypes.oneOfType([
+      PropTypes.arrayOf(PropTypes.node),
+      PropTypes.node
+    ]).isRequired,
+    left: PropTypes.bool
   }
 
   static defaultProps = {
diff --git a/src/components/Header.js b/src/components/Header.js
index 8b3bf19..3e30f13 100644
--- a/src/components/Header.js
+++ b/src/components/Header.js
@@ -1,8 +1,12 @@
-import React, {Component} from 'react';
-import '../stylesheets/globals.scss';
-import styles from './Header.scss';
+import React, { Component } from 'react'
+import PropTypes from 'prop-types'
+import '../stylesheets/globals.scss'
+import styles from './Header.scss'
 
 export default class Header extends Component {
+  static propTypes = {
+    header: PropTypes.string.isRequired
+  }
 
   render () {
     return (
diff --git a/src/components/Home.js b/src/components/Home.js
index c48853f..1a1c031 100644
--- a/src/components/Home.js
+++ b/src/components/Home.js
@@ -10,7 +10,7 @@ export default class Home extends Component {
     config: PropTypes.object.isRequired
   }
 
-  render () {  
+  render () {
     return (
       <div id={styles.coverPage} className={styles.coverPageFull} role="region" aria-label="Home page">
         <div id={styles.coverPageContent}>
diff --git a/src/components/Navbar.js b/src/components/Navbar.js
index 4bc25ba..4666c76 100644
--- a/src/components/Navbar.js
+++ b/src/components/Navbar.js
@@ -5,7 +5,6 @@ import '../stylesheets/globals.scss'
 import styles from './Navbar.scss'
 
 export default class Navbar extends Component {
-
   static propTypes = {
     config: PropTypes.object.isRequired
   }
diff --git a/src/components/Resume.js b/src/components/Resume.js
index ba2deb8..6ad9af8 100644
--- a/src/components/Resume.js
+++ b/src/components/Resume.js
@@ -10,7 +10,8 @@ import fm from 'front-matter'
 export default class About extends Component {
   static propTypes = {
     isLoading: PropTypes.bool.isRequired,
-    resume: PropTypes.string.isRequired
+    resume: PropTypes.string.isRequired,
+    config: PropTypes.object.isRequired
   }
 
   render () {
@@ -28,7 +29,7 @@ export default class About extends Component {
     }
 
     return (
-        <div>
+      <div>
         <Navbar config={this.props.config} />
         <Wrapper>
           <div className={`${contentStyle.content} ${style.column}`}>
diff --git a/src/components/SocialLinks.js b/src/components/SocialLinks.js
index 0f3f881..dd64dc2 100644
--- a/src/components/SocialLinks.js
+++ b/src/components/SocialLinks.js
@@ -21,7 +21,7 @@ export default class SocialLinks extends Component {
       const link = (
         <li key={key}>
           <a href={this.props.config.social[val]} role="link">
-            <i className={`fa fa-${val} ${this.props.home ? 'fa-3x': ''}`} aria-hidden="true" />
+            <i className={`fa fa-${val} ${this.props.home ? 'fa-3x' : ''}`} aria-hidden="true" />
             <span className="sr-only">{val}</span>
           </a>
         </li>
@@ -33,7 +33,7 @@ export default class SocialLinks extends Component {
     socialLinks.push(
       <li key={key}>
         <a href={`mailto:${this.props.config.email}`} role="link">
-          <i className={`fa fa-envelope-o ${this.props.home ? 'fa-3x': ''}`} aria-hidden="true" />
+          <i className={`fa fa-envelope-o ${this.props.home ? 'fa-3x' : ''}`} aria-hidden="true" />
           <span className="sr-only">e-mail</span>
         </a>
       </li>
@@ -42,11 +42,11 @@ export default class SocialLinks extends Component {
     const className = this.props.home ? styles['social-home'] : styles['social-navbar']
 
     return (
-        <div className={className} role="list">
-            <ul>
-            {socialLinks}
-            </ul>
-        </div>
+      <div className={className} role="list">
+        <ul>
+          {socialLinks}
+        </ul>
+      </div>
     )
   }
-}
\ No newline at end of file
+}
diff --git a/src/components/index.js b/src/components/index.js
index da046c9..a7f8b01 100644
--- a/src/components/index.js
+++ b/src/components/index.js
@@ -3,7 +3,6 @@ export { default as Blog } from './Blog'
 export { default as About } from './About'
 export { default as Post } from './Post'
 export { default as NotFoundPage } from './NotFoundPage'
-export { default as NotFoundWrapper } from './NotFoundWrapper'
 export { default as Spinner } from './Spinner'
 export { default as Header } from './Header'
 export { default as Wrapper } from './Wrapper'
diff --git a/src/containers/ContentContainer.js b/src/containers/ContentContainer.js
index 7c381e9..045c14d 100644
--- a/src/containers/ContentContainer.js
+++ b/src/containers/ContentContainer.js
@@ -21,14 +21,14 @@ export default class ContentContainer extends Component {
 
     this.state = {
       isLoading: !data,
-      type: data[0]['type'],
-      content: data[0]['data'],
+      type: data[0].type,
+      content: data[0].data,
       config: data[1]
-    }    
+    }
   }
 
   render () {
-    if (this.state.type == 'resume') {
+    if (this.state.type === 'resume') {
       return (
         <Resume
           isLoading={this.state.isLoading}
diff --git a/src/components/NotFoundWrapper.js b/src/containers/NotFoundContainer.js
similarity index 71%
rename from src/components/NotFoundWrapper.js
rename to src/containers/NotFoundContainer.js
index 902f863..1ec8a02 100644
--- a/src/components/NotFoundWrapper.js
+++ b/src/containers/NotFoundContainer.js
@@ -1,18 +1,18 @@
 import React, { Component } from 'react'
-import { Wrapper, NotFoundPage } from '.'
+import { Wrapper, NotFoundPage } from '../components'
 import PropTypes from 'prop-types'
 import '../stylesheets/globals.scss'
 
-export default class NotFoundWrapper extends Component {
+export default class NotFoundContainer extends Component {
   static propTypes = {
-    config: PropTypes.object.isRequired
+    staticContext: PropTypes.object.isRequired
   }
 
   constructor (props) {
     super(props)
-
     let data
 
+    // eslint-disable-next-line no-undef
     if (__isBrowser__) {
       data = window.__INITIAL_DATA__
       delete window.__INITIAL_DATA__
@@ -22,7 +22,7 @@ export default class NotFoundWrapper extends Component {
 
     this.state = {
       config: data[1]
-    }    
+    }
   }
 
   render () {
diff --git a/src/containers/index.js b/src/containers/index.js
index 26b9cd4..45aa0e9 100644
--- a/src/containers/index.js
+++ b/src/containers/index.js
@@ -1,2 +1,3 @@
 export { default as MainContainer } from './MainContainer'
 export { default as ContentContainer } from './ContentContainer'
+export { default as NotFoundContainer } from './NotFoundContainer'