Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: mise a jour de la date de fin de l'historique des indicateurs apprenants #3610

Merged
merged 11 commits into from
Mar 20, 2024

Conversation

nkrmr
Copy link
Collaborator

@nkrmr nkrmr commented Mar 20, 2024

Description

Cette PR corrige la fonction genererHistoriqueStatutFromApprenant pour assurer une gestion correcte de la date de fin lors de la génération de l'historique des statuts des apprenants. Le correctif implémente une logique permettant de choisir correctement entre la date de fin de période de formation et la date d'évaluation actuelle, afin de déterminer la date de fin effective à utiliser.

Détails du correctif

  • Le paramètre endDate a été renommé en evaluationEndDate pour clarifier son rôle dans la comparaison des dates.
  • Introduction d'une condition pour sélectionner la date de fin la plus appropriée, basée sur la comparaison entre la date de fin de la période de formation et la date d'évaluation.
  • La logique ajustée assure que la date de fin utilisée ne dépasse pas la fin de la période de formation, tout en restant flexible pour les cas où l'évaluation se fait avant la fin de cette période.

Raison du correctif

Ce correctif est nécessaire pour éviter les incohérences dans l'historique généré, où des statuts pourraient être erronément calculés au-delà de la période de formation effective. En assurant une sélection appropriée de la date de fin, ce correctif améliore la précision de l'historique des statuts et sa conformité aux périodes de formation définies.

@nkrmr nkrmr added this pull request to the merge queue Mar 20, 2024
Merged via the queue into master with commit 399bed5 Mar 20, 2024
5 checks passed
@nkrmr nkrmr deleted the feat/tm-822-2 branch March 20, 2024 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants