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.

Sign up for GitHub

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

Jump to bottom

Revalorisation des plafonds des ressources applicables Métropole - Allocations familiales 01/2018 #970

Merged
merged 7 commits into from
Apr 18, 2018
Merged

Revalorisation des plafonds des ressources applicables Métropole - Allocations familiales 01/2018 #970

merged 7 commits into from
Apr 18, 2018

Conversation

frtomas
Copy link
Contributor

@frtomas frtomas commented Apr 17, 2018

  • Évolution du système socio-fiscal.
  • Périodes concernées : à partir du 01/01/2018.
  • Zones impactées : parameters\prestations\prestations_familiales\af\modulation\
  • Détails :
    • Revalorisation des plafonds de ressources et de la majoration du plafond par enfant dans le calcul de l'Allocation Familiale

Ces changements :

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

guillett
guillett reviewed Apr 17, 2018
View reviewed changes
Copy link
Member

@guillett guillett left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Bonjour et merci beaucoup @frtomas pour cette contribution !

C'est super d'avoir ajouté les tests avec les modifications de la législation.

...tions/prestations_familiales/af/modulation/majoration_plafond_par_enfant_supplementaire.yaml
@@ -7,3 +7,6 @@ values:
value: 5595.0
2016-01-01:
value: 5617.0
2018-01-01:
value: 5628.0
Copy link
Member

Choose a reason for hiding this comment

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

Aurais-tu une référence législative pour ces revalorisations ?

Est-ce la revalorisation forfaitaire prévue dans l'article L161-25 du code de la Sécurité Sociale ?

Copy link
Contributor Author

@frtomas frtomas Apr 18, 2018 •
edited
Loading

Choose a reason for hiding this comment

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

@guillett J'ai commit l'ajout des références législatives, pour l'AF il s'agit précisément de cet article :
https://www.legifrance.gouv.fr/eli/arrete/2017/12/27/SSAS1736592A/jo/article_1
de "l’Arrêté du 27 décembre 2017 relatif au montant des plafonds de certaines prestations familiales..."

Copy link
Member

Choose a reason for hiding this comment

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

Merci beaucoup !

tests/formulas/af_2018.yaml
- id: "enfant2"
age: 10
output_variables:
af: 129.86 / (1 - 0.005)
Copy link
Member

Choose a reason for hiding this comment

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

J'ai l'impression que les montants sont ajustés de la CSG. Est-ce bien la raison de l'ajustement de / (1 - 0.005) ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Effectivement, je vais l'indiquer en commentaires dans les tests.

Copy link
Member

Choose a reason for hiding this comment

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

Encore merci !

guillett
guillett reviewed Apr 17, 2018
View reviewed changes
tests/formulas/af_2018.yaml
- id: "enfant4"
age: 5
output_variables:
af: 231.31 / (1 - 0.005) + 48.97
Copy link
Member

Choose a reason for hiding this comment

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

Pourrais-tu ajouter un commentaire pour expliquer les calculs réalisés pour la valeur attendue en af ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Je vais commenter le détail des calculs pour les rendre plus lisibles.

@guillett guillett mentioned this pull request Apr 17, 2018
Merged
@guillett
Copy link
Member

Merci @frtomas pour les ajouts et les clarifications.

J'attends les indications sur les valeurs attendues et ça devrait être bon pour merger.

@guillett
Copy link
Member

Super, merci @frtomas pour ces ajouts.

Dans ton dernier commit tu as ajouté des tests sur les ALs qui me paraissent hors du périmètre de cette PR.

Peut-on les sortir de cette PR et en discutant par ailleurs ?

@frtomas
Copy link
Contributor Author

frtomas commented Apr 18, 2018

Effectivement @guillett c’était une petite erreur de manipulation, ce fichier n'avait rien à faire dans cette PR.

guillett
guillett approved these changes Apr 18, 2018
View reviewed changes
Copy link
Member

@guillett guillett left a comment

Choose a reason for hiding this comment

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

Encore merci pour cette contribution !

J'ai fait un commentaire sur lequel il me parait intéressant de continuer à échanger.

...tions/prestations_familiales/af/modulation/majoration_plafond_par_enfant_supplementaire.yaml
@@ -7,3 +7,6 @@ values:
value: 5595.0
2016-01-01:
value: 5617.0
2018-01-01:
value: 5628.0
Copy link
Member

Choose a reason for hiding this comment

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

Merci beaucoup !

tests/formulas/af_2018.yaml
- id: "enfant2"
age: 10
output_variables:
af: 129.86 / (1 - 0.005)
Copy link
Member

Choose a reason for hiding this comment

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

Encore merci !

tests/formulas/af_2018.yaml
@@ -0,0 +1,1071 @@
- name: "Allocations familiales - Cas N°1"
Copy link
Member

Choose a reason for hiding this comment

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

Les numéros de cas de test ne permettent pas de comprendre ce qui est testé dans le test. Le fichier à ta disposition contient-il des informations complémentaires pertinentes ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Non, ce n'est pas une informations qui est précisée dans les cas de tests à ma disposition, je n'ai que les données en entrée et en sortie.

Copy link
Member

Choose a reason for hiding this comment

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

Ok, merci beaucoup pour ton retour.

@guillett guillett merged commit be900b6 into master Apr 18, 2018
@guillett guillett deleted the msa_reval_af branch April 18, 2018 15:26
@guillett guillett mentioned this pull request Apr 18, 2018
Closed
@jmdallais jmdallais mentioned this pull request Apr 19, 2018
Closed
@bonjourmauko bonjourmauko added contrib:msa Identification des sujets MSA kind:evolution labels Jul 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@guillett guillett guillett approved these changes

Assignees
No one assigned
Labels
contrib:msa Identification des sujets MSA kind:evolution
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@frtomas @guillett @bonjourmauko