From 1f65b3b7eb026a5cdc9e0b39fd7f1ddaabdac57d Mon Sep 17 00:00:00 2001 From: Oliver Eyton-Williams Date: Tue, 8 Dec 2020 16:53:02 +0100 Subject: [PATCH] fix: require project completion to claim certs (#40399) Co-authored-by: Kristofer Koishigawa Co-authored-by: Kristofer Koishigawa --- curriculum/getChallenges.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/curriculum/getChallenges.js b/curriculum/getChallenges.js index 845fb0c4e1..01399673ff 100644 --- a/curriculum/getChallenges.js +++ b/curriculum/getChallenges.js @@ -248,10 +248,13 @@ ${getFullPath('english')} // while the auditing is ongoing, we default to English for un-audited certs // once that's complete, we can revert to using isEnglishChallenge(fullPath) const useEnglish = lang === 'english' || !isAuditedCert(lang, superBlock); - const isCert = path.extname(filePath) === 'markdown'; + const isCert = path.extname(filePath) === '.markdown'; let challenge; if (isCert) { + // TODO: this uses the old parser to handle certifcates, but Markdown is a + // clunky way to store data, consider converting to YAML and removing the + // old parser. challenge = await (useEnglish ? parseMarkdown(getFullPath('english')) : parseTranslation(