Skip to content

Commit

Permalink
Merge pull request #2517 from codecrafters-io/stage-2-experiments
Browse files Browse the repository at this point in the history
End stage 2 experiment: Short Instructions vs Link to Code Examples
  • Loading branch information
andy1li authored Jan 6, 2025
2 parents 3a8b1d6 + 00873ab commit 382c0b6
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 9 deletions.
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<CoursePage::InstructionsCard @title="How to pass this stage" id="second-stage-tutorial-card" ...attributes>
<:content>
<div class="prose dark:prose-invert mb-5">
{{#if (and this.shouldShowShortInstructions (eq @repository.course.slug "shell"))}}
{{#if (eq @repository.course.slug "shell")}}
<div class="mb-5 prose-compact">
{{markdown-to-html @courseStage.shortInstructionsMarkdown}}
</div>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -83,10 +83,6 @@ export default class SecondStageTutorialCardComponent extends Component<Signatur
);
}

get shouldShowShortInstructions() {
return this.featureFlags.canSeeShortInstructionsForStage2;
}

get steps() {
return [
new ImplementSolutionStep(this.args.repository, this.implementSolutionStepIsComplete),
Expand Down
4 changes: 0 additions & 4 deletions app/services/feature-flags.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,10 +14,6 @@ export default class FeatureFlagsService extends Service {
return this.currentUser && (this.currentUser.isStaff || this.currentUser.isConceptAuthor);
}

get canSeeShortInstructionsForStage2() {
return this.currentUser?.isStaff || this.getFeatureFlagValue('can-see-short-instructions-for-stage-2') === 'test';
}

get currentUser() {
return this.authenticator.currentUser;
}
Expand Down

0 comments on commit 382c0b6

Please sign in to comment.