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

[TRA-12646] Incohérence entre les dates affichées dans l'Aperçu et les dates affichées dans le registre #3319

Merged
merged 11 commits into from
May 27, 2024

Conversation

GaelFerrand
Copy link
Collaborator

@GaelFerrand GaelFerrand commented May 14, 2024

Contexte

Certains BSDD en production ont une incohérence entre la date affichée dans l'Aperçu et la date affichée dans le registre. Cette incohérence survient pour des dates "edge-case" à cheval sur minuit, donc en fait le bug est général, mais ne se manifeste que pour un nombre extrêmement limité de dates (paramétrées par API, puisque parfaitement à minuit).

La cause du bug est que nous utilisons un code différent pour l'affichage des dates côté front et côté registre.

J'ai donc corrigé le bug en utilisant le même code.

Démo

demo_dates

Ticket Favro

Registre, erreur sur les dates de réception pour les BSDD DEEE avec éco-organisme

Copy link

sonarcloud bot commented May 16, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@GaelFerrand GaelFerrand changed the title [TRA-12646] Erreur sur les dates de réception pour les BSDD DEEE avec éco-organisme [TRA-12646] Incohérence entre les dates affichées dans l'Aperçu et les dates affichées dans le registre May 21, 2024
@GaelFerrand GaelFerrand merged commit b178e28 into dev May 27, 2024
17 checks passed
@GaelFerrand GaelFerrand deleted the tra-12646 branch May 27, 2024 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants