Skip to content

PASS IAE: correction du code empêchant l'envoi d'un PASS avant sa date de début#5666

Merged
xavfernandez merged 1 commit intomasterfrom
xfernandez/fix_local_date_computation
Feb 21, 2025
Merged

PASS IAE: correction du code empêchant l'envoi d'un PASS avant sa date de début#5666
xavfernandez merged 1 commit intomasterfrom
xfernandez/fix_local_date_computation

Conversation

@xavfernandez
Copy link
Contributor

@xavfernandez xavfernandez commented Feb 21, 2025

🤔 Pourquoi ?

now est en UTC donc now.date() & localdate(now) ne renvoie pas la même chose aux alentours de minuit.

Donc entre minuit & 1h (et bientôt entre minuit & 2h), le temps qu'UTC rattrape CEST, on sélectionne des PASS démarrant aujourd'hui et qui se font tout de suite retoquer avant d'être réessayé 5 minutes plus tard.

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@xavfernandez xavfernandez added modifié Modifié dans le changelog. bug labels Feb 21, 2025
@xavfernandez xavfernandez self-assigned this Feb 21, 2025
Copy link
Member

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ça faisait un moment que je la laissais sous le tapis, merci !

Copy link
Contributor

@EwenKorr EwenKorr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est un beau cas pratique de ce qu'on a vu en journée tech !
Les dates et les logs 🤓

@xavfernandez xavfernandez added this pull request to the merge queue Feb 21, 2025
Merged via the queue into master with commit 47673d7 Feb 21, 2025
15 checks passed
@xavfernandez xavfernandez deleted the xfernandez/fix_local_date_computation branch February 21, 2025 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants