(issue #516) clean up commented codes and todo usages

config/webpack.config.dev.js

@@ -117,10 +117,6 @@ module.exports = {
   module: {
     strictExportPresence: true,
     rules: [
-      // TODO: Disable require.ensure as it's not a standard language feature.
-      // We are waiting for https://github.com/facebookincubator/create-react-app/issues/2176.
-      // { parser: { requireEnsure: false } },
-
       // First, run the linter.
       // It's important to do this before Babel processes the JS.
       {

config/webpack.config.prod.js

@@ -116,10 +116,6 @@ module.exports = {
   module: {
     strictExportPresence: true,
     rules: [
-      // TODO: Disable require.ensure as it's not a standard language feature.
-      // We are waiting for https://github.com/facebookincubator/create-react-app/issues/2176.
-      // { parser: { requireEnsure: false } },
-
       // First, run the linter.
       // It's important to do this before Babel processes the JS.
       {

src/scenes/home/families/jumboQuote/jumboQuote.js

@@ -13,8 +13,6 @@ class JumboQuote extends Component {
     };
   }
 
-  // TODO: Ditch bluebird promise for axios to drop a dependency
-  // TODO: Is there a need to move the quotes to a back-end API endpoint, or should we just bail on the promise-based API request idea for this data?
   componentDidMount() {
     Promise.try(
       () => quotes.quotes[Math.floor(Math.random() * quotes.quotes.length)]

src/scenes/home/history/history.js

@@ -2,7 +2,6 @@ import React, { Component } from 'react';
 import Section from 'shared/components/section/section';
 import QuoteBanner from 'shared/components/quoteBanner/quoteBanner';
 import Timeline from './timeline/timeline';
-// import styles from './history.css';
 
 class History extends Component {
   render() {

src/scenes/home/history/timelineEvent/timelineEvent.js

@@ -3,8 +3,6 @@ import PropTypes from 'prop-types';
 import styles from './timelineEvent.css';
 
 class TimelineEvent extends Component {
-  // TODO: Force eventTitle to Title case with Regex
-
   render() {
     return (
       <div className={styles.eventContainer}>

src/scenes/home/landing/whatWeDo/whatWeDo.js

@@ -17,29 +17,21 @@ class WhatWeDo extends Component {
             image={image1}
             title={content.items[0].title}
             cardText={content.items[0].body}
-            // buttonText={content.items[0].buttonText}
-            // link={content.items[0].linkTo}
           />
           <ImageCard
             image={image2}
             title={content.items[1].title}
             cardText={content.items[1].body}
-            // buttonText={content.items[1].buttonText}
-            // link={content.items[1].linkTo}
           />
           <ImageCard
             image={image3}
             title={content.items[2].title}
             cardText={content.items[2].body}
-            // buttonText={content.items[2].buttonText}
-            // link={content.items[2].linkTo}
           />
           <ImageCard
             image={image4}
             title={content.items[3].title}
             cardText={content.items[3].body}
-            // buttonText={content.items[3].buttonText}
-            // link={content.items[3].linkTo}
           />
         </div>
       </Section>

src/shared/components/join/join.js

@@ -1,5 +1,4 @@
 import React from 'react';
-// import PropTypes from 'prop-types';
 import Section from 'shared/components/section/section';
 import LinkButton from 'shared/components/linkButton/linkButton';
 import styles from './join.css';

src/shared/utils/cookieHelper.js

@@ -47,8 +47,6 @@ const validToken = (token) => {
   const jwt = jwt_decode(token);
   const currentTime = new Date().getTime() / 1000;
 
-  // TODO we could validate the signature as well (issue#281)
-
   if (currentTime > jwt.exp) { return false; }
 
   return true;