From 48532d6ccb14e2430e1cd07e1da641f60d055562 Mon Sep 17 00:00:00 2001
From: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Date: Fri, 22 May 2020 18:04:25 +0200
Subject: [PATCH] fix: hide isHidden challenges from hotkey nav

---
 client/gatsby-node.js                       | 6 ++++--
 client/utils/gatsby/challengePageCreator.js | 5 ++---
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/client/gatsby-node.js b/client/gatsby-node.js
index c22bd824bc..be1f216913 100644
--- a/client/gatsby-node.js
+++ b/client/gatsby-node.js
@@ -122,10 +122,12 @@ exports.createPages = function createPages({ graphql, actions, reporter }) {
         }
 
         // Create challenge pages.
-        result.data.allChallengeNode.edges.forEach(
-          createChallengePages(createPage)
+        const challengeEdges = result.data.allChallengeNode.edges.filter(
+          ({ node: { isHidden } }) => !isHidden
         );
 
+        challengeEdges.forEach(createChallengePages(createPage));
+
         // Create intro pages
         result.data.allMarkdownRemark.edges.forEach(edge => {
           const {
diff --git a/client/utils/gatsby/challengePageCreator.js b/client/utils/gatsby/challengePageCreator.js
index b749f48166..1a1fec145a 100644
--- a/client/utils/gatsby/challengePageCreator.js
+++ b/client/utils/gatsby/challengePageCreator.js
@@ -67,12 +67,11 @@ exports.createChallengePages = createPage => ({ node }, index, thisArray) => {
     required = [],
     template,
     challengeType,
-    id,
-    isHidden
+    id
   } = node;
   // TODO: challengeType === 7 and isPrivate are the same, right? If so, we
   // should remove one of them.
-  if (challengeType === 7 || isHidden) {
+  if (challengeType === 7) {
     return null;
   }