Améliore les conditions d'éligibilité à la PPA pour les couples avec étudiant #1361
Conversation
|
@guillett : juste pour savoir est-ce essentiel de créer une nouvelle variable ? |
|
La création d'une nouvelle variable n'est pas nécessaire mais très appréciable pour les investigations :) Je la rajoute à la |
|
Done @benjello. |
There was a problem hiding this comment.
Hello @guillett, merci pour ce refactoring. Il me semble très clair, par contre je n'arrive pas à trouver la référence exacte pour le plancher, pourrais-tu m'aider ?
|
C'est dans les articles indiqués à |
| definition_period = MONTH | ||
|
|
||
| def formula(individu, period, parameters): | ||
| P = parameters(period) |
There was a problem hiding this comment.
Remplacer P par une chaîne de caractère en minuscule.
Proposition bancale: parameters_at_period ?
There was a problem hiding this comment.
Le P = parameters(period) semble être utilisé sous cette forme un peu partout dans ce fichier et ailleurs quel problème cela vient résoudre ?
There was a problem hiding this comment.
Les majuscules c'est pour les variables d'environnement et autres variables globales.
Et oui il faudrait les virer partout :-)
Raison de plus pour ne pas en rajouter.
There was a problem hiding this comment.
Je suis d'accord avec @benjello sur le fond, mais il ne me semble pas bloquant pour accepter le PR. On pourra faire un PR pour corriger cela comment on a fait pour les u"".
There was a problem hiding this comment.
Je suis d'accord avec @maukoquiroga qui est d'accord avec moi :-)
|
Et virer les |
|
Merci pour ta contribution @guillett ! |
|
Il faudrait dé-publier cette version car elle est cassée en vectorielle. |
48.3.3 #1361
prestations/minima_sociaux/ppa.Aujourd'hui un couple avec un étudiant dont les revenus d'activité sont inférieurs au plancher est tout de même considéré comme éligible à la PPA si son conjoint n'est pas étudiant.
Cette PR vérifie que son conjoint a des revenus d'activité non nuls.