From a06eaec5d4227649519227d15448a069e8558dd0 Mon Sep 17 00:00:00 2001
From: Kamran Ahmed <kamranahmed.se@gmail.com>
Date: Sun, 1 Dec 2019 16:16:45 +0400
Subject: [PATCH] Add summary pages to landscape

---
 components/detailed-roadmap/index.js          |  6 ++-
 scripts/roadmaps-meta.js                      |  6 ++-
 static/sitemap.xml                            | 36 +++++++++++++++
 storage/roadmaps.json                         | 45 ++++++++++++++++---
 .../{summary.md => landscape/0-Summary.md}    |  0
 .../{summary.md => landscape/0-Summary.md}    |  0
 .../{summary.md => landscape/0-Summary.md}    |  0
 .../{summary.md => landscape/0-Summary.md}    |  0
 .../{summary.md => landscape/0-Summary.md}    |  0
 9 files changed, 84 insertions(+), 9 deletions(-)
 rename storage/roadmaps/1-frontend/{summary.md => landscape/0-Summary.md} (100%)
 rename storage/roadmaps/2-backend/{summary.md => landscape/0-Summary.md} (100%)
 rename storage/roadmaps/3-devops/{summary.md => landscape/0-Summary.md} (100%)
 rename storage/roadmaps/4-fullstack/{summary.md => landscape/0-Summary.md} (100%)
 rename storage/roadmaps/5-qa/{summary.md => landscape/0-Summary.md} (100%)

diff --git a/components/detailed-roadmap/index.js b/components/detailed-roadmap/index.js
index 52a212c7d..1e8255610 100644
--- a/components/detailed-roadmap/index.js
+++ b/components/detailed-roadmap/index.js
@@ -27,8 +27,12 @@ const DetailedRoadmap = ({ roadmap }) => {
         <h3>{ groupTitle }</h3>
         <ul>
           { sidebar[groupTitle].map(page => {
+            const isActivePage = page.url === currentPage.url;
+            // e.g. /frontend should mark `/frontend/summary` as active
+            const isSummaryPage = page.url === `${currentPage.url}/summary`;
+
             return (
-              <li className={classNames({ active: page.url === currentPage.url })}>
+              <li className={classNames({ active: isActivePage || isSummaryPage })}>
                 <a href={ page.url }>
                   <span className="bullet"></span>
                   { page.title }
diff --git a/scripts/roadmaps-meta.js b/scripts/roadmaps-meta.js
index 17782f95a..1c88fdd40 100644
--- a/scripts/roadmaps-meta.js
+++ b/scripts/roadmaps-meta.js
@@ -14,6 +14,10 @@ const roadmapsMeta = roadmapDirs.reduce((metaAcc, roadmapDirName) => {
   const roadmapDir = path.join(ROADMAPS_PATH, roadmapDirName);
   const roadmapMeta = require(path.join(roadmapDir, 'meta.json'));
 
+  // We can't use the absolute path in the build e.g. ~/Users/user/where-build-is-running/storage
+  // So, we remove it and use the path relative to storage directory
+  const summaryFilePath = path.join(roadmapDir.replace(STORAGE_PATH, ''), '/landscape/0-Summary.md')
+
   const contributors = exec(`git log --pretty=format:"%an%x09" ${roadmapDir} | uniq`)
     .toString()
     .split('\n')
@@ -72,7 +76,7 @@ const roadmapsMeta = roadmapDirs.reduce((metaAcc, roadmapDirName) => {
       contributorsCount: contributorNames.length,
       contributorsUrl: `/${roadmapSlug}/contributors`,
       url: `/${roadmapSlug}`,
-      path: path.join(roadmapDir.replace(STORAGE_PATH, ''), '/summary.md'),
+      path: summaryFilePath,
       sidebar,
     },
   ];
diff --git a/static/sitemap.xml b/static/sitemap.xml
index 9ad8ebd94..b2098cf1a 100644
--- a/static/sitemap.xml
+++ b/static/sitemap.xml
@@ -6,6 +6,18 @@
               <lastmod>2019-11-30T10:14:37.750Z</lastmod>
               <priority>1.0</priority>
           </url>
+<url>
+              <loc>https://roadmap.sh/frontend/contributors</loc>
+              <changefreq>monthly</changefreq>
+              <lastmod>2019-12-01T11:38:19.737Z</lastmod>
+              <priority>1.0</priority>
+          </url>
+<url>
+              <loc>https://roadmap.sh/frontend/summary</loc>
+              <changefreq>monthly</changefreq>
+              <lastmod>2019-11-30T10:14:37.750Z</lastmod>
+              <priority>1.0</priority>
+          </url>
 <url>
               <loc>https://roadmap.sh/frontend/junior</loc>
               <changefreq>monthly</changefreq>
@@ -78,6 +90,12 @@
               <lastmod>2019-11-30T10:14:31.255Z</lastmod>
               <priority>1.0</priority>
           </url>
+<url>
+              <loc>https://roadmap.sh/backend/summary</loc>
+              <changefreq>monthly</changefreq>
+              <lastmod>2019-11-30T10:14:31.255Z</lastmod>
+              <priority>1.0</priority>
+          </url>
 <url>
               <loc>https://roadmap.sh/backend/junior</loc>
               <changefreq>monthly</changefreq>
@@ -102,6 +120,12 @@
               <lastmod>2019-11-30T10:24:46.555Z</lastmod>
               <priority>1.0</priority>
           </url>
+<url>
+              <loc>https://roadmap.sh/devops/summary</loc>
+              <changefreq>monthly</changefreq>
+              <lastmod>2019-11-30T10:24:46.555Z</lastmod>
+              <priority>1.0</priority>
+          </url>
 <url>
               <loc>https://roadmap.sh/devops/junior</loc>
               <changefreq>monthly</changefreq>
@@ -126,12 +150,24 @@
               <lastmod>2019-11-27T20:14:37.663Z</lastmod>
               <priority>1.0</priority>
           </url>
+<url>
+              <loc>https://roadmap.sh/fullstack/summary</loc>
+              <changefreq>monthly</changefreq>
+              <lastmod>2019-11-27T20:14:37.663Z</lastmod>
+              <priority>1.0</priority>
+          </url>
 <url>
               <loc>https://roadmap.sh/qa</loc>
               <changefreq>monthly</changefreq>
               <lastmod>2019-11-27T20:14:50.558Z</lastmod>
               <priority>1.0</priority>
           </url>
+<url>
+              <loc>https://roadmap.sh/qa/summary</loc>
+              <changefreq>monthly</changefreq>
+              <lastmod>2019-11-27T20:14:50.558Z</lastmod>
+              <priority>1.0</priority>
+          </url>
 <url>
               <loc>https://roadmap.sh/guides/design-patterns-for-humans</loc>
               <changefreq>monthly</changefreq>
diff --git a/storage/roadmaps.json b/storage/roadmaps.json
index e87449412..f83c36004 100644
--- a/storage/roadmaps.json
+++ b/storage/roadmaps.json
@@ -17,7 +17,7 @@
     "contributorsCount": 1,
     "contributorsUrl": "/frontend/contributors",
     "url": "/frontend",
-    "path": "/roadmaps/1-frontend/summary.md",
+    "path": "/roadmaps/1-frontend/landscape/0-Summary.md",
     "sidebar": {
       "_others": [
         {
@@ -27,6 +27,11 @@
         }
       ],
       "landscape": [
+        {
+          "url": "/frontend/summary",
+          "title": "Summary",
+          "path": "/roadmaps/1-frontend/landscape/0-Summary.md"
+        },
         {
           "url": "/frontend/junior",
           "title": "Junior",
@@ -99,9 +104,14 @@
     "contributorsCount": 1,
     "contributorsUrl": "/backend/contributors",
     "url": "/backend",
-    "path": "/roadmaps/2-backend/summary.md",
+    "path": "/roadmaps/2-backend/landscape/0-Summary.md",
     "sidebar": {
       "landscape": [
+        {
+          "url": "/backend/summary",
+          "title": "Summary",
+          "path": "/roadmaps/2-backend/landscape/0-Summary.md"
+        },
         {
           "url": "/backend/junior",
           "title": "Junior",
@@ -137,9 +147,14 @@
     "contributorsCount": 1,
     "contributorsUrl": "/devops/contributors",
     "url": "/devops",
-    "path": "/roadmaps/3-devops/summary.md",
+    "path": "/roadmaps/3-devops/landscape/0-Summary.md",
     "sidebar": {
       "landscape": [
+        {
+          "url": "/devops/summary",
+          "title": "Summary",
+          "path": "/roadmaps/3-devops/landscape/0-Summary.md"
+        },
         {
           "url": "/devops/junior",
           "title": "Junior",
@@ -170,8 +185,16 @@
     "contributorsCount": 1,
     "contributorsUrl": "/fullstack/contributors",
     "url": "/fullstack",
-    "path": "/roadmaps/4-fullstack/summary.md",
-    "sidebar": {}
+    "path": "/roadmaps/4-fullstack/landscape/0-Summary.md",
+    "sidebar": {
+      "landscape": [
+        {
+          "url": "/fullstack/summary",
+          "title": "Summary",
+          "path": "/roadmaps/4-fullstack/landscape/0-Summary.md"
+        }
+      ]
+    }
   },
   {
     "title": "QA Engineer",
@@ -185,7 +208,15 @@
     "contributorsCount": 1,
     "contributorsUrl": "/qa/contributors",
     "url": "/qa",
-    "path": "/roadmaps/5-qa/summary.md",
-    "sidebar": {}
+    "path": "/roadmaps/5-qa/landscape/0-Summary.md",
+    "sidebar": {
+      "landscape": [
+        {
+          "url": "/qa/summary",
+          "title": "Summary",
+          "path": "/roadmaps/5-qa/landscape/0-Summary.md"
+        }
+      ]
+    }
   }
 ]
\ No newline at end of file
diff --git a/storage/roadmaps/1-frontend/summary.md b/storage/roadmaps/1-frontend/landscape/0-Summary.md
similarity index 100%
rename from storage/roadmaps/1-frontend/summary.md
rename to storage/roadmaps/1-frontend/landscape/0-Summary.md
diff --git a/storage/roadmaps/2-backend/summary.md b/storage/roadmaps/2-backend/landscape/0-Summary.md
similarity index 100%
rename from storage/roadmaps/2-backend/summary.md
rename to storage/roadmaps/2-backend/landscape/0-Summary.md
diff --git a/storage/roadmaps/3-devops/summary.md b/storage/roadmaps/3-devops/landscape/0-Summary.md
similarity index 100%
rename from storage/roadmaps/3-devops/summary.md
rename to storage/roadmaps/3-devops/landscape/0-Summary.md
diff --git a/storage/roadmaps/4-fullstack/summary.md b/storage/roadmaps/4-fullstack/landscape/0-Summary.md
similarity index 100%
rename from storage/roadmaps/4-fullstack/summary.md
rename to storage/roadmaps/4-fullstack/landscape/0-Summary.md
diff --git a/storage/roadmaps/5-qa/summary.md b/storage/roadmaps/5-qa/landscape/0-Summary.md
similarity index 100%
rename from storage/roadmaps/5-qa/summary.md
rename to storage/roadmaps/5-qa/landscape/0-Summary.md