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

Remplace année par âge pour pass colo #2443

Merged
merged 2 commits into from
Mar 10, 2025
Merged

Remplace année par âge pour pass colo #2443

merged 2 commits into from
Mar 10, 2025

Conversation

sylvainipp
Copy link
Contributor

  • Amélioration technique.
    Remplace le critère d'année de naissance par un critère d'âge directement écrit dans la législation pour le pass colo.

Ces changements (effacez les lignes ne correspondant pas à votre cas) :

  • Corrigent ou améliorent un calcul déjà existant.

Quelques conseils à prendre en compte :

Et surtout, n'hésitez pas à demander de l'aide ! :)

Copy link
Contributor

@jenovateurs jenovateurs left a comment

Choose a reason for hiding this comment

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

Bonjour @sylvainipp,

Merci pour cette contribution et de m'avoir notifié avec mon collègue Simon.

Sur le fond, je suis entièrement d'accord. À la lecture de Légifrance, la modélisation est bonne.

Sauf que je vous invite à lire la PR d'origine sur laquelle nous avons débattu de ce sujet : #2299

Je pense qu'on peut toujours redemander la confirmation auprès de jeune.gouv et/ou vérifier si les autres simulateurs continuent d'adopter le même principe de la date de naissance.

Il faut toutefois mettre à jour l'année de naissance si on reste dans le précédent cas de figure.

Au plaisir de poursuivre la conversation lundi prochain, car nous sommes en mode maintenance avec un temps réduit (uniquement les lundis).

@sylvainipp
Copy link
Contributor Author

Bonjour ! Merci pour le lien vers la première PR. J'ai l'impression que dans les deux modélisations nous ne pouvons pas bien prendre en compte les enfants qui ont droit au pass colo car ils ne l'ont pas demandé l'année précédente. De plus, la modélisation de cette PR est strictement équivalente au fait d'être né en 2013 pour 2024 et 2014 pour 2025 (je regarde l'âge au premier janvier et pas l'âge courant), avec l'avantage d'être actualisé automatiquement et d'utiliser la valeur inscrite dans la loi. Si je comprends que mettre l'âge directement puisse causer des simulations fausses, j'ai l'impression que l'âge au premier janvier permet de réconcilier Légifrance et les résultats des autres simulateurs.

Copy link
Contributor

@jenovateurs jenovateurs left a comment

Choose a reason for hiding this comment

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

Merci pour le retour, j'ai mieux compris la démarche.
Après test, tout est bon pour moi (sous réserve de mettre à jour le changelog, setup…).
Merci encore pour la contribution.

@sylvainipp sylvainipp merged commit ec21cb2 into master Mar 10, 2025
38 checks passed
@sylvainipp sylvainipp deleted the age_colo branch March 10, 2025 16:30
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