-
Notifications
You must be signed in to change notification settings - Fork 101
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
Implémentation de l’abattement sur le revenu de substitution pour l’ASS #1326
Conversation
fix #995 |
@guillett Tu confirmes bien qu'il y avait une erreur dans le calcul existant et que cette PR le corrige ? Cf. le test Mes Aides impacté par cette PR. |
Merci @Morendil pour la mention, merci @mtifarine pour la PR et merci @monbocal pour l'issue initial. Je n'ai pas l'expertise mais après avoir rapidement regardé je pense que cette PR va dans le bon sens. |
@guillett @mtifarine Dans ce cas si c'est OK pour tout le monde je propose de merger aujourd'hui. |
… sur les revenus d'individu
6036eb6
to
bd3dcaa
Compare
@mtifarine Les tests ne passent plus après le rebase, je fais l'hypothèse que cette PR qui était basée sur la version 42.3.0 n'intégrait pas la revalorisation dans #1328 - je mets à jour le test avec les valeurs actualisées. (ETA: j'ai supprimé ce commit suite aux échanges ci-dessous) |
Merci @Morendil, |
5f7d9b5
to
bd3dcaa
Compare
@mtifarine Et ça demande plus de manipulations: il faut répercuter la période d'évaluation sur les différentes périodes des variables d'entrée; j'ai essayé pour le dernier test ("Abattement des revenus quand le revenu d'individu et du conjoint sont interrompus") et je n'arrive pas à le faire passer en décalant toutes les périodes. Je préfère te laisser faire l'ajustement pour la revalorisation, c'est plus sûr. |
fad3680
to
98ecb81
Compare
Merci @mtifarine ! |
prestations/minima_sociaux/ass
.