528-article-default-image (#529)

* fixed bug where VTT files were being picked up as article files

* use default image path when article does not have a YouTube video code

Author: marshmallowrobot

scripts/generate_learning_path_markdown.js

@@ -21,11 +21,11 @@ const getYouTubeCode = (section, articleNumber) => {
   return ''
 }
 
-const getYouTubeImage = (youTubeCode) => {
+const getArticleImage = (youTubeCode) => {
   if (youTubeCode) {
     return `https://img.youtube.com/vi/${youTubeCode}/mqdefault.jpg`
   }
-  return ''
+  return 'images/learn/LP-article-default.png'
 }
 
 (async () => {
@@ -77,7 +77,7 @@ const getYouTubeImage = (youTubeCode) => {
         const frontMatter = {
           title: articleTitle,
           contributors,
-          image: getYouTubeImage(youtubeCode),
+          image: getArticleImage(youtubeCode),
           featured: weight === 1,
           weight,
           youtubeCode

scripts/get_article_files.js

@@ -4,7 +4,7 @@ const { join } = require('path')
 module.exports = getArticleFiles = (path) => {
   return fs.readdirSync(path).reduce((articles, filename) => {
     const filePath = join(path, filename)
-    if (filePath.match(/\d\d/) && !filePath.includes('-script.asciidoc')) {
+    if (filePath.match(/\d\d[-\w]+(?<!script)\.asciidoc/)) {
       return [...articles, {
         filePath,
         asciiDoc: fs.readFileSync(filePath, 'utf-8')
@@ -13,4 +13,4 @@ module.exports = getArticleFiles = (path) => {
       return articles
     }
   }, [])
-}
+}