Skip to content

Conversation

@tpommellet-pass
Copy link
Contributor

@tpommellet-pass tpommellet-pass commented Dec 4, 2025

🎯 Related Ticket or 🔧 Changes Made

Ticket Jira

Capture d’écran 2025-12-04 à 15 31 30

@github-actions github-actions bot added api Pull requests that update /api code ticket labels Dec 4, 2025
Copy link
Contributor

@rpaoloni-pass rpaoloni-pass left a comment

Choose a reason for hiding this comment

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

Si les deux sont le même jours est ce que ça a du sens d'afficher les deux ?

@tpommellet-pass
Copy link
Contributor Author

Si les deux sont le même jours est ce que ça a du sens d'afficher les deux ?

@rpaoloni-pass je ne suis pas sûr de bien comprendre la question, peut être que mon screenshot peut induire en erreur : on ne s'intéresse ici qu'à l'affichage au sein d'une seule colonne. Si pour une colonne, la période démarre et termine le même jour (par exemple le 5 décembre 2025), on ne souhaite pas afficher 05/12/2025 -> 05/12/2025 mais juste 05/12/2025.

Si deux colonnes ont le même valeur, par exemple la diffusion sur l'espace partenaire et la date de l'évènement, alors on souhaite bien afficher les deux dates : cela permet aux opérateurs d'être certain d'avoir bien planifier la valorisation thématique. Si l'on souhaite néanmoins modifier ce comportement alors cela fera l'objet d'un autre ticket !

@rpaoloni-pass
Copy link
Contributor

@tpommellet-pass dans ce cas pourquoi tu ne compare pas simplement les deux string générées ? ça serait plus simple non ? si c'est la même tu n'en retourne qu'une seule

@tpommellet-pass
Copy link
Contributor Author

@tpommellet-pass dans ce cas pourquoi tu ne compare pas simplement les deux string générées ? ça serait plus simple non ? si c'est la même tu n'en retourne qu'une seule

@rpaoloni-pass oui j'ai hésité entre les deux solutions, j'ai pas d'opinion très tranché sur la question. Ma version permet de ne pas calculer le string end si les dates sont égales, mais le gain de perf est très marginale, donc si tu penses qu'on perd en lisibilité alors ça me va totalement de comparer les deux strings générées !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api Pull requests that update /api code ticket

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants