remove remaining console warnings

ShMcK · ShMcK · commit 77eafc1e004d · 2016-03-04T09:21:33.000-08:00
diff --git a/lib/components/page/hint.js b/lib/components/page/hint.js
@@ -29,7 +29,6 @@ var default_1 = (function (_super) {
     default_1.prototype.render = function () {
         var _a = this.props, task = _a.task, hintPosition = _a.hintPosition, nextHint = _a.nextHint, prevHint = _a.prevHint;
         var hints = task && task.hints ? task.hints : null;
-        console.log(hintPosition, hints, nextHint);
         if (hintPosition < 0 || !hints || !hints.length) {
             return React.createElement("div", null);
         }
diff --git a/lib/components/progress/progress.js b/lib/components/progress/progress.js
@@ -85,7 +85,7 @@ exports.default = function (_a) {
             'isActive': isActive
         }), secondaryText: chapter.description, secondaryTextLines: chapter.description.length > 35 ? 2 : 1, initiallyOpen: chapterIndex === 0, leftIcon: chapter.completed ? React.createElement(AllCompleted, null) : null, primaryTogglesNestedList: chapterIndex === position.chapter && !chapter.completed, nestedItems: chapter.pages.map(function (page, pageIndex) {
             var itemPosition = { chapter: chapterIndex, page: pageIndex };
-            return React.createElement(ProgressPage, {page: page, itemPosition: itemPosition, position: position});
+            return React.createElement(ProgressPage, {key: 'c' + chapterIndex + 'p' + pageIndex, page: page, itemPosition: itemPosition, position: position});
         })});
     }))));
 };
diff --git a/lib/reducers/alert/alert.js b/lib/reducers/alert/alert.js
@@ -42,16 +42,14 @@ function alertReducer(alert, action) {
                 };
                 return current;
             }
-            else {
-                statusBarAlert.style.color = '#9DA5B4';
-                current = {
-                    message: result.msg,
-                    open: true,
-                    action: 'note',
-                    duration: result.duration || 2500
-                };
-                return current;
-            }
+            statusBarAlert.style.color = '#9DA5B4';
+            current = {
+                message: result.msg,
+                open: true,
+                action: 'note',
+                duration: result.duration || 2500
+            };
+            return current;
         case Type.PAGE_COMPLETE:
             return {
                 message: "Page " + (action.payload.position.page + 1) + " Complete",
diff --git a/src/components/progress/progress.tsx b/src/components/progress/progress.tsx
@@ -83,7 +83,7 @@ export default ({progress, position}) => (
           primaryTogglesNestedList={chapterIndex === position.chapter && !chapter.completed}
           nestedItems={chapter.pages.map((page: CR.Page, pageIndex: number) => {
             const itemPosition = {chapter: chapterIndex, page: pageIndex};
-            return <ProgressPage page={page} itemPosition={itemPosition} position={position}/>;
+            return <ProgressPage key={'c' + chapterIndex + 'p' + pageIndex} page={page} itemPosition={itemPosition} position={position}/>;
          })}/>;
     })}
     </List>
diff --git a/src/reducers/alert/alert.ts b/src/reducers/alert/alert.ts
@@ -43,38 +43,37 @@ export default function alertReducer(alert = defaultAlert, action: CR.Action): C
           duration: result.duration || 2500
         };
         return current;
-      } else {
-        // Alert
-        statusBarAlert.style.color = '#9DA5B4';
-        current = {
-          message: result.msg,
-          open: true,
-          action: 'note',
-          duration: result.duration || 2500
-        };
-        return current;
       }
-      case Type.PAGE_COMPLETE:
+      // Alert
+      statusBarAlert.style.color = '#9DA5B4';
+      current = {
+        message: result.msg,
+        open: true,
+        action: 'note',
+        duration: result.duration || 2500
+      };
+      return current;
+    case Type.PAGE_COMPLETE:
       return {
         message: `Page ${action.payload.position.page + 1} Complete`,
         open: true,
         action: 'pass',
         duration: 2000
       };
-      case Type.CHAPTER_COMPLETE:
-        return {
-          message: `Chapter ${action.payload.chapter + 1} Complete`,
-          open: true,
-          action: 'pass',
-          duration: 2000
-        };
-      case Type.PROJECT_COMPLETE:
-        return {
-          message: 'Tutorial Complete',
-          open: true,
-          action: 'pass',
-          duration: 2000
-        };
+    case Type.CHAPTER_COMPLETE:
+      return {
+        message: `Chapter ${action.payload.chapter + 1} Complete`,
+        open: true,
+        action: 'pass',
+        duration: 2000
+      };
+    case Type.PROJECT_COMPLETE:
+      return {
+        message: 'Tutorial Complete',
+        open: true,
+        action: 'pass',
+        duration: 2000
+      };
     default:
       return alert;
   }

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,6 @@ var default_1 = (function (_super) {`
`29`	`29`	`default_1.prototype.render = function () {`
`30`	`30`	`var _a = this.props, task = _a.task, hintPosition = _a.hintPosition, nextHint = _a.nextHint, prevHint = _a.prevHint;`
`31`	`31`	`var hints = task && task.hints ? task.hints : null;`
`32`		`- console.log(hintPosition, hints, nextHint);`
`33`	`32`	`if (hintPosition < 0 \|\| !hints \|\| !hints.length) {`
`34`	`33`	`return React.createElement("div", null);`
`35`	`34`	`}`