fix courses list filtering and titles
This commit is contained in:
committed by
Benedikt Rötsch
parent
28764d2386
commit
60d2c02ce7
@@ -14,7 +14,7 @@ router.get('/', async function (req, res, next) {
|
|||||||
} catch (e) {
|
} catch (e) {
|
||||||
console.log('Error ', e)
|
console.log('Error ', e)
|
||||||
}
|
}
|
||||||
res.render('courses', { title: 'Courses', categories, courses })
|
res.render('courses', { title: `All Courses (${courses.length})`, categories, courses })
|
||||||
})
|
})
|
||||||
|
|
||||||
/* GET courses listing. */
|
/* GET courses listing. */
|
||||||
@@ -22,13 +22,15 @@ router.get('/categories/:category', async function (req, res, next) {
|
|||||||
// we get all the entries with the content type `course`
|
// we get all the entries with the content type `course`
|
||||||
let courses = []
|
let courses = []
|
||||||
let categories = []
|
let categories = []
|
||||||
|
let activeCategory = ''
|
||||||
try {
|
try {
|
||||||
courses = await getCoursesByCategory(req.params.category, req.query.locale, req.query.api)
|
courses = await getCoursesByCategory(req.params.category, req.query.locale, req.query.api)
|
||||||
categories = await getCategories()
|
categories = await getCategories()
|
||||||
|
activeCategory = categories.find((category) => category.sys.id === req.params.category)
|
||||||
} catch (e) {
|
} catch (e) {
|
||||||
console.log('Error ', e)
|
console.log('Error ', e)
|
||||||
}
|
}
|
||||||
res.render('courses', { title: 'Courses', categories, courses })
|
res.render('courses', { title: `${activeCategory.fields.title} (${courses.length})`, categories, courses })
|
||||||
})
|
})
|
||||||
|
|
||||||
/* GET course detail. */
|
/* GET course detail. */
|
||||||
|
|||||||
@@ -58,7 +58,7 @@ exports.getCoursesByCategory = (category, locale = 'en-US', api = `cda`) => {
|
|||||||
const client = api === 'cda' ? cdaClient : cpaClient
|
const client = api === 'cda' ? cdaClient : cpaClient
|
||||||
return client.getEntries({
|
return client.getEntries({
|
||||||
content_type: 'course',
|
content_type: 'course',
|
||||||
'fields.category.sys.contentType.sys.id': category,
|
'fields.categories.sys.id': category,
|
||||||
locale,
|
locale,
|
||||||
include: 10
|
include: 10
|
||||||
})
|
})
|
||||||
|
|||||||
Reference in New Issue
Block a user