transform about into a landing page and generalize code (#12)

routes/about.js

@@ -1,9 +1,12 @@
 const express = require('express')
+const { getLandingPage } = require('../services/contentful')
+const { catchErrors } = require('../handlers/errorHandlers')
 const router = express.Router()
 
-/* GET about page. */
-router.get('/', function (req, res, next) {
-  res.render('about', { title: 'About' })
-})
+/* GET the about landing page. */
+router.get('/', catchErrors(async function (req, res, next) {
+  const landingPage = await getLandingPage('about', req.query.locale, req.query.api)
+  res.render('landingPage', { title: 'About', landingPage })
+}))
 
 module.exports = router

routes/index.js

@@ -3,10 +3,10 @@ const { getLandingPage } = require('../services/contentful')
 const { catchErrors } = require('../handlers/errorHandlers')
 const router = express.Router()
 
-/* GET home page. */
+/* GET the home landing page. */
 router.get('/', catchErrors(async function (req, res, next) {
-  const landingPage = await getLandingPage(req.query.locale, req.query.api)
-  res.render('index', { title: 'Contentful University', landingPage })
+  const landingPage = await getLandingPage('home', req.query.locale, req.query.api)
+  res.render('landingPage', { title: 'Contentful University', landingPage })
 }))
 
 module.exports = router

services/contentful.js

@@ -31,11 +31,10 @@ exports.getCourses = assert((locale = 'en-US', api = `cda`) => {
     .then((response) => response.items)
 }, 'Course')
 
-exports.getLandingPage = (locale = 'en-US', api = `cda`) => {
-  // our Home page is fully configureable via Contentful
+exports.getLandingPage = (slug, locale = 'en-US', api = `cda`) => {
+  // Landing pages like the home or about page are fully controlable via Contentful.
   const client = api === 'cda' ? cdaClient : cpaClient
-  // TODO slug should be renamed to `contentful-the-example-app` or something ....
-  return client.getEntries({content_type: 'landingPage', locale, 'fields.slug': 'contentful-university', include: 10})
+  return client.getEntries({content_type: 'landingPage', locale, 'fields.slug': slug, include: 10})
     .then((response) => response.items[0])
 }
 

views/about.pug

@@ -1,6 +0,0 @@
-extends layout
-
-block content
-  .layout-centered
-    h1= title
-    p Welcome to #{title}

views/index.pug

@@ -1,18 +0,0 @@
-extends layout
-
-include mixins/_moduleCopy
-include mixins/_moduleHeroImage
-include mixins/_moduleHighlightedCourse
-
-block content
-  .modules-container
-    each module in landingPage.fields.contentModules
-      case module.sys.contentType.sys.id
-        when 'landingPageModuleCopy'
-          +moduleCopy(module)
-
-        when 'landingPageModuleHeroImage'
-          +moduleHeroImage(module)
-
-        when 'landingPageModuleHighlightedCourse'
-          +moduleHighlightedCourse(module, module.fields.course)

views/landingPage.pug

@@ -0,0 +1,18 @@
+extends layout
+
+include mixins/_moduleCopy
+include mixins/_moduleHeroImage
+include mixins/_moduleHighlightedCourse
+
+block content
+  .modules-container
+      each module in landingPage.fields.contentModules
+        case module.sys.contentType.sys.id
+          when 'landingPageModuleCopy'
+            +moduleCopy(module)
+
+          when 'landingPageModuleHeroImage'
+            +moduleHeroImage(module)
+
+          when 'landingPageModuleHighlightedCourse'
+            +moduleHighlightedCourse(module, module.fields.course)