fix(code-reviews): dédupliquer les membres sur la page détail d'un groupe#136
Merged
Conversation
…oupe Zone01 peut renvoyer plusieurs lignes de progression par étudiant pour un même projet/groupe (retries) : la page /code-reviews/[promoId]/group/[groupId] mappait les progressions brutes et affichait chaque membre en double (constaté en prod sur le groupe 10746). On déduplique par login — même règle que buildProjectGroups, déjà appliquée sur la liste des groupes — en gardant la première ligne et le premier grade non nul, sur des copies (les progressions viennent d'un cache partagé). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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.
Bug (prod)
Sur https://hub.zone01normandie.org/code-reviews/1226/group/10746, chaque membre du groupe apparaît deux fois dans le panneau « Infos → Membres ».
Cause
Zone01 peut renvoyer plusieurs lignes de progression par étudiant pour un même projet/groupe (retries). La liste des groupes passe par
buildProjectGroups()qui déduplique par login, mais la page détail mappaitprogressions.filter(...)brut → une ligne rendue par progression.Vérifié en prod : la table
audit_resultsest saine (1 ligne par étudiant) — le bug est purement d'affichage.Fix
Déduplication par login dans la page détail (même règle que
buildProjectGroups), en conservant la première ligne et le premiergradenon nul, sur des copies pour ne pas muter le cache partagé des progressions.🤖 Generated with Claude Code