I apologize that my anal nature got the better of me. Nearly all of the files apart from generate_learning_path_markdown.js contain capitalization fixes.
As for generate_learning_path_markdown.js, I ended up adding a couple of small functions that
don't assume the article has corresponding video URL entries in the URL config (this was the main bug)
article title parsing looks for both .asciidoc and .md syntaxes
There are now enough article-related functions in this file that I'm open to moving them into their own module. It could be overkill though, since all uses are still contained in this script.
I also wonder whether the hard-coded YouTube thumbnail URL (see getYouTubeImage()) should be moved to the front-end template instead of here?
And as always, appreciate any feedback ya'll have on the code too. Its been at least... two years? since I've had hands on code (eep!).
One silly question I have: what happened to semicolon line endings in JS? Are they just considered hipster throwback punctuation now?
I apologize that my anal nature got the better of me. Nearly all of the files apart from
generate_learning_path_markdown.js
contain capitalization fixes.As for
generate_learning_path_markdown.js
, I ended up adding a couple of small functions that.asciidoc
and.md
syntaxes There are now enough article-related functions in this file that I'm open to moving them into their own module. It could be overkill though, since all uses are still contained in this script.I also wonder whether the hard-coded YouTube thumbnail URL (see
getYouTubeImage()
) should be moved to the front-end template instead of here?And as always, appreciate any feedback ya'll have on the code too. Its been at least... two years? since I've had hands on code (eep!).
One silly question I have: what happened to semicolon line endings in JS? Are they just considered hipster throwback punctuation now?