Agile611/the-example-app-nodejs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(routes): avoid 404 by duplicating course route

Browse Source
This commit is contained in:
Benedikt Rötsch
2017-10-06 14:31:14 +02:00
committed by Benedikt Rötsch
parent 229b218980
commit 7e49f18db1
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
routes/courses.js
View File

@@ -22,7 +22,7 @@ router.get('/categories/:category', catchErrors(async function (req, res, next)
try { try {
categories = await getCategories() categories = await getCategories()
activeCategory = categories.find((category) => category.fields.slug === req.params.category) activeCategory = categories.find((category) => category.fields.slug === req.params.category)
courses = await getCoursesByCategory(activeCategory.sys.id, req.query.locale, req.query.api) courses = await getCoursesByCategory(activeCategory.sys.id, req.query.locale, req.query.api)
} catch (e) { } catch (e) {
console.log('Error ', e) console.log('Error ', e)
} }
@@ -30,7 +30,7 @@ router.get('/categories/:category', catchErrors(async function (req, res, next)
})) }))
/* GET course detail. */ /* GET course detail. */
router.get('/:slug', catchErrors(async function (req, res, next) { const courseRoute = catchErrors(async function (req, res, next) {
let course = await getCourse(req.params.slug, req.query.locale, req.query.api) let course = await getCourse(req.params.slug, req.query.locale, req.query.api)
const lessons = course.fields.lessons const lessons = course.fields.lessons
const lessonIndex = lessons.findIndex((lesson) => lesson.fields.slug === req.params.lslug) const lessonIndex = lessons.findIndex((lesson) => lesson.fields.slug === req.params.lslug)
@@ -41,7 +41,9 @@ router.get('/:slug', catchErrors(async function (req, res, next) {
visitedLessons = [...new Set(visitedLessons)] visitedLessons = [...new Set(visitedLessons)]
res.cookie('visitedLessons', visitedLessons, { maxAge: 900000, httpOnly: true }) res.cookie('visitedLessons', visitedLessons, { maxAge: 900000, httpOnly: true })
res.render('course', {title: course.fields.title, course, lesson, lessons, lessonIndex, visitedLessons}) res.render('course', {title: course.fields.title, course, lesson, lessons, lessonIndex, visitedLessons})
})) })
router.get('/:slug', courseRoute)
router.get('/:slug/lessons', courseRoute)
/* GET course lesson detail. */ /* GET course lesson detail. */
router.get('/:cslug/lessons/:lslug', catchErrors(async function (req, res, next) { router.get('/:cslug/lessons/:lslug', catchErrors(async function (req, res, next) {