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 de la PAJE au 1er avril 2018 #994

Merged
merged 6 commits into from
Sep 13, 2018
Merged

Revalorisation de la PAJE au 1er avril 2018 #994

merged 6 commits into from
Sep 13, 2018

Conversation

frtomas
Copy link
Contributor

@frtomas frtomas commented May 25, 2018

  • Évolution du système socio-fiscal.
  • Périodes concernées : à partir du 01/04/2018.
  • Zones impactées :
    • parameters/prestations/prestations_familiales/af
    • parameters/prestations/prestations_familiales/paje
  • Détails :
    • Revalorise les plafonds de ressources et les montants de la PAJE en date du 01/04/2018.

Ces changements :

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

@frtomas frtomas force-pushed the msa_reval_paje_01_04_2018 branch 2 times, most recently from cbb9150 to 7a1bac2 Compare May 29, 2018 08:43
@fpagnoux fpagnoux requested a review from guillett June 1, 2018 16:52
@guillett
Copy link
Member

guillett commented Jun 5, 2018

@frtomas je ne vois pas de tests dans cette PR mais il y a pas mal de changements structurels.

@frtomas
Copy link
Contributor Author

frtomas commented Jun 5, 2018

@guillett On est ici dans le même cas que ce dont parle @jmdallais dans la PR #983 :

Bonjour @guillett

Effectivement un des commits supprime les tests. Ce sont des tests à ce jour KO, non immédiatement liés au périmètre traité dans l'issue (revalorisation de paramètres de dépenses), et relatifs à la fiabilisation du calcul de la prestation.

Je m'explique :
Lorsque nous avons fait l'état des lieux MSA / DINSIC fin octobre, début novembre 2017, tu nous as mis en visibilité les limitations / retours usagers connus pour certaines prestations.

Les PR concernant

les AL ( #983 )
la PPA ( #984 )
le RSA (#985 )
sont concernées par ce nécessaire travail de fiabilisation.

Il a été décidé avec la DSS que ces travaux de fiabilisation serait menés post PNDS V2 et pas mis sur le chemin critique de ce jalon (le 13 juin). Leur priorisation est en cours (@ThibaultCCMSA, @JenniferTelep, @Lashchuk, DSS, DHUP, DGCS,...).

A noter par ailleurs que potentiellement l'AAH, la PAJE et la CMUc-ACS sont dans le même cas (PR à venir ce jour par @frtomas ).

Espérant avoir répondu à tes interrogations.

Si besoin de plus d'infos, n'hésites pas.

@frtomas frtomas force-pushed the msa_reval_paje_01_04_2018 branch 2 times, most recently from 53c07a3 to e86650e Compare June 6, 2018 14:17
@monbocal
Copy link
Contributor

monbocal commented Jun 8, 2018

A noter que cette PR modifie également la BMAF, ce qui permet d'avoir des calculs à jour pour les bourses - merci !

benjello
benjello requested changes Jun 22, 2018
View reviewed changes
Copy link
Member

@benjello benjello left a comment

Choose a reason for hiding this comment

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

J'ai indiqué quelques changements à effectuer.
Je n'ai pas reviewé le reste du refactoring.

.../prestations/prestations_familiales/paje/base/apres_2018/majoration_biact_parent_isoles.yaml Outdated
@@ -0,0 +1,6 @@
description: Majoration pour biactifs et isolés
reference: ipp
Copy link
Member

Choose a reason for hiding this comment

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

La référence est noté ici comme venant de l'IPP. Or ce tag est lié au parsing des barèmes IPP donc je pense, sauf si la valeur a été extraite des barèmes IPP, qu'il faut le changer en openfisca.

...ance/parameters/prestations/prestations_familiales/paje/base/apres_2018/plaf_tx_par_enf.yaml Outdated
@@ -0,0 +1,6 @@
description: Majoration du plafond par enfant
reference: ipp
Copy link
Member

Choose a reason for hiding this comment

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

Idem

...meters/prestations/prestations_familiales/paje/base/apres_2018/plafond_ressources_0_enf.yaml Outdated
@@ -0,0 +1,6 @@
description: Plafond de ressources
reference: ipp
Copy link
Member

Choose a reason for hiding this comment

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

Idem

.../prestations/prestations_familiales/paje/base/apres_2018/taux_majoration_2_premiers_enf.yaml Outdated
@@ -0,0 +1,6 @@
description: Majoration du plafond pour les 2 premiers enfants
reference: ipp
Copy link
Member

Choose a reason for hiding this comment

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

Idem

...restations/prestations_familiales/paje/base/apres_2018/taux_majoration_3eme_enf_et_plus.yaml Outdated
@@ -0,0 +1,6 @@
description: Majoration du plafond à partir du 3ème enfant
reference: ipp
Copy link
Member

Choose a reason for hiding this comment

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

Idem

@jmdallais jmdallais added the contrib:msa Identification des sujets MSA label Jul 9, 2018
@alexsegura alexsegura mentioned this pull request Jul 23, 2018
Merged
guillett
guillett previously requested changes Aug 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.

@frtomas quand pourras-tu regarder les retours de @benjello et ajouter des tests (ou ajouter du contexte pour expliquer pourquoi il n'y en a pas) ?

@frtomas
Copy link
Contributor Author

frtomas commented Aug 17, 2018 •
edited
Loading

@guillett c'est en cours avec une passe de fiabilisation de la PAJE pour comprendre d'où vienne tous les écarts des tests dont nous disposons.

@frtomas frtomas added contrib:msa Identification des sujets MSA and removed contrib:msa Identification des sujets MSA labels Aug 22, 2018
@frtomas frtomas requested a review from guillett August 29, 2018 08:18
@frtomas
Copy link
Contributor Author

frtomas commented Aug 29, 2018

@guillet j'ai ajouté les tests et donné une réponse à @benjello (je ne sais pas si la réponse l'a satisfait)

@frtomas
Copy link
Contributor Author

frtomas commented Sep 4, 2018

@guillett suite au dernier message de Jean-marie, nous devons continuer les validations de ces PRs, je suis donc à ta disposition pour voir ce qu'il reste à changer/corriger sur celle-ci.

Cordialement.

@benjello
Copy link
Member

@frtomas : désolé pour la longueur du temps de réponse

@guillett guillett dismissed their stale review September 10, 2018 16:18

Je fais confiance à @benjello. Disponibilité limitée.

@frtomas frtomas force-pushed the msa_reval_paje_01_04_2018 branch from da63313 to 55cdf8f Compare September 11, 2018 07:59
@benjello benjello mentioned this pull request Sep 12, 2018
Merged
@bfabre01
Copy link

@frtomas @benjello @guillett : cette PR est-elle prête à être mergée ? Ou bien reste-il des choses ? On aurait besoin de cette actualisation avec @claireleroy .

@benjello
Copy link
Member

@frtomas, @guillet il faut tenir compte de #1110 et puis c'est bon je pense.

@guillett
Copy link
Member

@frtomas vous mergez ?

@frtomas frtomas force-pushed the msa_reval_paje_01_04_2018 branch from 55cdf8f to 1bfcbe0 Compare September 13, 2018 12:03
@frtomas frtomas merged commit 2cf4e27 into master Sep 13, 2018
@bonjourmauko bonjourmauko deleted the msa_reval_paje_01_04_2018 branch September 13, 2018 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benjello benjello benjello approved these changes

@guillett guillett Awaiting requested review from guillett

@Anna-Livia Anna-Livia Awaiting requested review from Anna-Livia

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.
9 participants
@frtomas @guillett @monbocal @benjello @bfabre01 @bonjourmauko @fpagnoux @jmdallais @marchand-laetitia-msa