Skip to content

fix-daily_defis + roulette

BADSI MASSI requested to merge fix-daily_defis into develop

Description

Correction d'un bug où les parties jouées en solo contre les bots incrémentaient à tort la progression des défis du jour qui sont censés être exclusivement multijoueurs.

Lien vers la tâche

Type de changement

  • Feature — nouvelle fonctionnalité
  • Fix — correction de bug
  • Refactor — modification du code sans changement fonctionnel
  • Docs — documentation uniquement
  • Chore — maintenance (deps, config, etc.)
  • Hotfix — correction urgente en production

Modifications principales

  • "server/src/dailyChallenges/dailyChallenge.service.ts" : Ajout d'une vérification isBotGame dans les fonctions incrementMultiplayerPlayCount et markWinWithPair pour bloquer la progression si la partie est contre les bots.

  • "server/src/sockets/game.gateway.ts" : Mise à jour de la fonction recordMultiPlayerStats pour passer isPracticeBotGameId(game.id) lors de l'appel aux services des défis.

Comment tester

  • Faire une partie contre un bot et une partie contre un joueur et voir si le défi s'incrémente de 1 ou pas.

Checklist

  • Code relu et conforme aux conventions du projet
  • Tests ajoutés ou mis à jour si nécessaire
  • Documentation mise à jour (si applicable)
  • Pipeline CI vert
  • Pas de régression détectée

Impact potentiel

  • Breaking change — migration nécessaire
  • Performance — impacts à surveiller
  • Sécurité — audit recommandé

Reviewers

  • Backend (@azra, @soheil)
  • Frontend (@mohamed, @yigit)
  • Sécurité (@linda, @massi)
  • BDD (@elhadj)
  • Revue générale (@aguech)
  • Validation chef de projet (@omar)
Edited by BADSI MASSI

Merge request reports