openfisca-france icon indicating copy to clipboard operation
openfisca-france copied to clipboard

Revalorisation du seuil de non recouvrement et ajout de références

Open mtifarine opened this issue 3 years ago • 8 comments

  • Changement mineur.

  • Périodes concernées : à partir du 01/01/2021.

  • Zones impactées :

    • parameters/prestations_sociales/aides_logement/allocations_logement/autres/nr_seuil.yaml
    • parameters/prestations_sociales/prestations_familiales/prestations_generales/cf/cf_plaf/plafond_ressources_0_enfant.yaml
    • tests/formulas/rsa.yaml
  • Détails :

    • Revalorisation du seuil de non recouvrement de l'AL.
    • Correction de la date des 2 cas de tests RSA.

mtifarine avatar May 25 '21 16:05 mtifarine

Merci @sandcha de votre retour! J'ai pris en compte les retours, pourriez-vous vérifier si c'est bon ?

mtifarine avatar Aug 24 '21 07:08 mtifarine

Rebase en cours, comme évoqué en réunion, en raison du nombre de conflicting files détecté et du temps écoulé.

sandcha avatar Sep 27 '21 07:09 sandcha

@mtifarine : en fait, on a déjà introduit côté IPP les revalorisations qui était l'objet de cette PR, qui date de mai. Voir les PR #1673 et #1668. D'où forcément les problèmes de rebase... Il faut donc changer le titre de cette PR ainsi que son descriptif, car elle n'catualise plus ce qu'elle était censée actualiser à la base. Et bien vérifier ce que cette PR modifie à l'issue du rebase. Car du coup, la proba d'erreur dans le rebase est relativement importante. Pour les revalorisations, il est important je pense de ne pas laisser une PR non-mergée aussi longtemps. Sinon, ça fait doublon. cc @sandcha

bfabre01 avatar Sep 28 '21 14:09 bfabre01

bonjour @benjello , @sandcha , @mtifarine , @frtomas dans la mesure où il n'y a plus de discussion sur cette PR est il possible de la merger s'il vous plait ? merci

tomy-isabelle avatar Dec 10 '21 11:12 tomy-isabelle

@robinguill @sandcha : est-ce encore à merger dans master ? Merci de fermer si ce n'est plus le cas.

benjello avatar Jun 28 '23 11:06 benjello

@benjello nous sommes en train de revoir toutes les PR ouvertes non merge ou close, celle ci revalorise le seuil de non recouvrement sur l'AL avec une nouvelle valeur pour 2021 pas encore présente sur le master et me semble donc encore d'actualité.

robinguill avatar Jun 29 '23 12:06 robinguill

@robinguill est-ce que je peux me permettre de rebaser cette branche car le bug qui apparaît dans les tests semble avoir été résolu sur Core ?

benjello avatar Jun 29 '23 13:06 benjello

Reste-t-il des points à discuter sur cette PRs ? Il reste une review déjà prise en compte mais non marquée à résolu @sandcha mais en dehors de ça, tout semble ok ?

frtomas avatar Oct 24 '23 07:10 frtomas