feat: enable new curriculum (#44183)
* feat: use legacy flag chore: reorder challenges fix: linter revert: server change feat: unblock new editor fix: proper order fix: 0-based order fix: broke the order feat: move tribute certification to its own block feat: split the old projects block into 4 fix: put all blocks in order chore: add intro text refactor: use block, not blockName in query fix: project progress indicator * fix: reorder new challenges/certs * fix: reorder legacy challenges * fix: reintroduce legacy certs * feat: add showNewCurriculum flag to env * chore: forgot sample.env * feat: use feature flag for display * fix: rename meta + dirs to match new blocks * fix: add new blocks to help-category-map * fix: update completion-modal for new GQL schema * test: duplicate title/id errors -> warnings * fix: update completion-modal to new GQL schema Mk2 * chore: re-order metas (again) * fix: revert super-block-intro changes The intro needs to show both legacy and new content. We need to decide which pages are created, rather than than what a page shows when rendered. * feat: move upcoming curriculum into own superblock * fix: handle one certification with two superBlocks * fix: remove duplicated intros * fix: remove duplicate projects from /settings * fix: drop 'two' from Responsive Web Design Two * chore: rename slug suffix from two to v2 * feat: control display of new curriculum * feat: control project paths shown on /settings * fix: use new project order for /settings This does mean that /settings will change before the release, but I don't think it's serious. All the projects are there, just not in the legacy order. * fix: claim/show cert button * chore: remove isLegacy Since we have legacy superblocks, we don't currently need individual blocks to be legacy * test: fix utils.test * fix: verifyCanClaim needs certification If Shaun removes the cert claim cards, maybe we can remove this entirely * fix: add hasEditableBoundaries flags where needed * chore: remove isUpcomingChange * chore: v2 -> 22 Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
This commit is contained in:
committed by
GitHub
parent
304773d62b
commit
8614db7a32
@ -228,6 +228,14 @@ async function buildChallenges({ path: filePath }, curriculum, lang) {
|
||||
}
|
||||
const { meta } = challengeBlock;
|
||||
const isCert = path.extname(filePath) === '.yml';
|
||||
// TODO: there's probably a better way, but this makes sure we don't build any
|
||||
// of the new curriculum when we don't want it.
|
||||
if (
|
||||
process.env.SHOW_NEW_CURRICULUM !== 'true' &&
|
||||
superBlock === 'responsive-web-design-22'
|
||||
) {
|
||||
return;
|
||||
}
|
||||
const challenge = isCert
|
||||
? await createCertification(challengesDir, filePath, lang)
|
||||
: await createChallenge(challengesDir, filePath, lang, meta);
|
||||
@ -307,8 +315,18 @@ ${getFullPath('english')}
|
||||
challenge.block = dasherize(blockName);
|
||||
challenge.hasEditableBoundaries = !!hasEditableBoundaries;
|
||||
challenge.order = order;
|
||||
const superOrder = getSuperOrder(superBlock);
|
||||
const superOrder = getSuperOrder(superBlock, {
|
||||
showNewCurriculum: process.env.SHOW_NEW_CURRICULUM === 'true'
|
||||
});
|
||||
if (superOrder !== null) challenge.superOrder = superOrder;
|
||||
/* Since there can be more than one way to complete a certification (using the
|
||||
legacy curriculum or the new one, for instance), we need a certification
|
||||
field to track which certification this belongs to. */
|
||||
// TODO: generalize this to all superBlocks
|
||||
challenge.certification =
|
||||
superBlock === 'responsive-web-design-22'
|
||||
? 'responsive-web-design'
|
||||
: superBlock;
|
||||
challenge.superBlock = superBlock;
|
||||
challenge.challengeOrder = challengeOrder;
|
||||
challenge.isPrivate = challenge.isPrivate || isPrivate;
|
||||
|
Reference in New Issue
Block a user