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

Corrections sur les paramètres liés à l'AAH et au CAAH #1353

Merged
merged 5 commits into from
Jul 4, 2019

Conversation

ManonLger
Copy link
Contributor

@ManonLger ManonLger commented Jul 2, 2019

  • Amélioration technique.
  • Périodes concernées : toutes.
  • Zones impactées :
    • openfisca_france/model/prestations/minima_sociaux/aah.py
    • openfisca_france/parameters/prestations/minima_sociaux/aah
    • openfisca_france/parameters/prestations/minima_sociaux/caah
  • Détails :
    • Supprime un doublon entre les paramètres aah.mva et caah.majoration_vie_autonome (on garde le second).
    • Supprime un doublon entre les paramètres caah.cpltx et caah.taux_du_montant_mensuel_du_complement_aux_adultes_handicape_2 (on renomme en caah.taux_montant_complement_ressources).
    • Renomme le paramètre aah.majoration_du_plafond_pour_un_couple en aah.majoration_plafond_couple.
    • Renomme le paramètre aah.tx_plaf_supp en aah.majoration_plafond_personne_a_charge.
    • Renomme le paramètre aah.plafond_de_ressources_en_multiple_du_montant_de_base en aah.plafond_ressources.

Fixes #1344

@ManonLger
Copy link
Contributor Author

En réponse à cette issue : #1344
@Morendil j'ai fait quelques autres modifs au passage, mais j'ai bien séparé les commits, donc on peut ne pas les faire si on estime que ça fait trop de changements par rapport à ce que ça apporte (surtout le dernier commit).

@ManonLger ManonLger requested a review from Morendil July 2, 2019 12:44
@Morendil
Copy link
Contributor

Morendil commented Jul 2, 2019

Merci @ManonLger ! Je regarderai en détail demain

@Morendil
Copy link
Contributor

Morendil commented Jul 4, 2019

@ManonLger Merci! J'ai requalifié la PR en "breaking change" car nous appliquons SemVer strictement, à part ça tout est OK.

@Morendil Morendil merged commit 5f21302 into master Jul 4, 2019
@Morendil Morendil deleted the fix-doublon-mva branch July 4, 2019 10:03
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.

Doublon pour la référence de la MVA
2 participants