Skip to content

fix(code-reviews): remonter le vrai statut du groupe dans /api/pending#131

Merged
makcimerrr merged 1 commit into
mainfrom
fix/pending-status-label
Jul 15, 2026
Merged

fix(code-reviews): remonter le vrai statut du groupe dans /api/pending#131
makcimerrr merged 1 commit into
mainfrom
fix/pending-status-label

Conversation

@makcimerrr

Copy link
Copy Markdown
Owner

Contexte

La gate d'auditabilité de /api/code-reviews/pending utilise déjà canAuditGroup() (statut finished OU audit) depuis #130. Mais le champ status renvoyé dans la réponse était resté codé en dur à 'finished'.

Effet

Un groupe soumis en statut audit apparaissait bien dans la file (comportement correct) mais étiqueté « finished » dans l'UI.

Fix

Propagation de group.status via groupData (+ status: string dans le type de la Map). Aucun changement fonctionnel de filtrage — purement le label affiché.

  • npx tsc --noEmit : ✅ clean

🤖 Generated with Claude Code

Le champ `status` de la file d'audits en attente était codé en dur à
'finished', alors que la gate utilise déjà canAuditGroup() (finished OU
audit). Un groupe soumis en `audit` s'affichait donc étiqueté "finished".
On propage désormais group.status via groupData.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
admin-dashboard Building Building Preview, Comment Jul 15, 2026 1:11pm

@github-actions github-actions Bot added the bug Something isn't working label Jul 15, 2026
@makcimerrr
makcimerrr merged commit c2cffd8 into main Jul 15, 2026
15 of 16 checks passed
@makcimerrr
makcimerrr deleted the fix/pending-status-label branch July 15, 2026 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant