chore(landing): the story mode moves to the end of the roadmap - #409
Merged
Merged
Conversation
Decided with @Thierry on 1 October 2026: the story mode (learn by following a character, each lesson a scene) starts only once the main track is finished. The landing roadmap lists it last, under "Après le parcours principal", and ROADMAP.md describes it as Phase 12. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Guide du réviseur (réduit pour les petites PR)Guide du réviseurLa PR déplace systématiquement le mode histoire narratif des travaux de gamification à court terme vers une phase finale de la feuille de route, après le parcours d’apprentissage principal, met à jour le texte de la page d’accueil et documente la décision ainsi que les contraintes initiales de mise en œuvre. Modifications au niveau des fichiers
Conseils et commandesInteraction avec Sourcery
Personnaliser votre expérienceAccédez à votre tableau de bord pour :
Obtenir de l’aide
Original review guide in EnglishReviewer's guide (collapsed on small PRs)Reviewer's GuideThe PR consistently moves the narrative story mode from near-term gamification work to a final roadmap phase after the main learning track, updates the landing-page wording, and documents the decision and initial implementation constraints. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
Bonjour - j’ai trouvé 1 problème
Invite pour les agents IA
Veuillez traiter les commentaires de cette revue de code :
## Commentaires individuels
### Commentaire 1
<location path="src/app/data/landingContent.ts" line_range="179" />
<code_context>
+ {
+ group: 'Après le parcours principal',
+ items: [
+ 'Mode histoire : suivre un personnage, chaque leçon devient une scène',
+ ],
+ },
</code_context>
<issue_to_address>
**nitpick :** Le nouveau groupe « Après le parcours principal » ne contient qu’un seul élément, ce qui contredit la documentation de `RoadmapGroup`, selon laquelle chaque pilier contient 2 à 4 fonctionnalités. Le commentaire décrit désormais de manière inexacte une structure introduite par cette modification.
**Correction suggérée :** Mettez à jour la documentation pour autoriser les groupes à élément unique, ou ajoutez une autre fonctionnalité du mode histoire afin que le groupe respecte l’invariant documenté de 2 à 4 éléments.
</issue_to_address>Sourcery est gratuit pour les projets open source : si vous appréciez nos revues, pensez à les partager ✨
Original comment in English
Hey - I've found 1 issue
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location path="src/app/data/landingContent.ts" line_range="179" />
<code_context>
+ {
+ group: 'Après le parcours principal',
+ items: [
+ 'Mode histoire : suivre un personnage, chaque leçon devient une scène',
+ ],
+ },
</code_context>
<issue_to_address>
**nitpick:** The new “Après le parcours principal” group contains only one item, contradicting the `RoadmapGroup` documentation that each pillar contains 2–4 features. The comment now misdescribes a structure introduced by this change.
**Suggested fix:** Update the documentation to allow single-item groups, or add another story-mode feature so the group satisfies the documented 2–4-item invariant.
</issue_to_address>Sourcery on #409: a roadmap group holds 2 to 4 items. The second one is the pilot ROADMAP.md already plans (one module before the rest). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Decided with @Thierry on 1 October 2026: the story mode (learn the terminal by following a character, each lesson a scene) is a large project that starts only once the main track is finished. The landing page listed it among the next Gamification items.
What
landingContent.ts: "Mode histoire narratif" leaves "Gamification & UX" and becomes the last group, "Après le parcours principal", worded as what it is ("suivre un personnage, chaque leçon devient une scène").docs/ROADMAP.md: new "Phase 12 — Story mode (after the main track)", with the open question of who draws.Checks
🤖 Generated with Claude Code
Résumé par Sourcery
Reporter le mode histoire jusqu’à la fin du parcours d’apprentissage principal.
Améliorations :
Documentation :
Original summary in English
Résumé par Sourcery
Reporter le mode histoire jusqu’à la fin du parcours d’apprentissage principal.
Améliorations :
Documentation :
Original summary in English
Summary by Sourcery
Defer the story mode until after the main learning track is complete.
Enhancements:
Documentation: