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

Update divers paramètres de l'IR pour les périodes récentes (2015-2017) #1017

Merged
merged 14 commits into from
Jun 28, 2018

Conversation

claireleroy
Copy link
Contributor

@claireleroy claireleroy commented Jun 25, 2018

Rapatriement des changements de législation de l'IR depuis 2015 à partir des infos des barèmes IPP.

  • Évolution du système socio-fiscal.
  • Périodes concernées : à partir du 01/01/2015.
  • Zones impactées : parameters\impot_revenu.
  • Détails :
    • Ajoute les paramètres de l'impôt sur les revenus 2017
    • Mets à jour les paramètres non actualisés de l'impôt sur les revenus 2015-2016

Ces changements :

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

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.

le champ reference sert ici à déterminer la source des paramètres datant de l'époque de la fusion automatique.
Même si je ne suis pas sûr que l'on le réutilisera et qu'il reste utilisable en l'état, il faut peut-être se contenter de garder les références législatives dans les barèmes IPP sachant qu'à terme on les récupérera. Mais je n'ai pas d'avis définitif @fpagnoux @Anna-Livia @sandcha @maukoquiroga

@claireleroy
Copy link
Contributor Author

@benjello je vire les références hors ipp donc ? ou j'attends d'autres retours sur la question ?
(idéalement on aimerais merger tout cela assez rapidement pour le transférer sur CASD)

@benjello
Copy link
Member

@ClaireLeroyIPP : tu peux déjà envoyer cela sur le CASD. Au pire il y aura des commits de nettoyage en plus de ceux-ci. Moi je nettoierais les références BoFiP si on les a ailleurs.

@benjello benjello requested a review from sandcha June 25, 2018 19:00
@fpagnoux
Copy link
Member

Je serais plutôt pour garder les références législatives, quitte à merger quand on importera celles de l'IPP :).

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.

Super taf !

@guillett
Copy link
Member

👏

@claireleroy
Copy link
Contributor Author

@Anna-Livia @sandcha qu'en pensez-vous ? est-ce que ça vous parait prêt à merger ?

@claireleroy
Copy link
Contributor Author

Oups je crois que j'ai demandé une review à ceux qui sont le moins disponible en ce moment ..
Du coup @guillett @maukoquiroga est-ce que vous pensez que ça peut être mergé ?

Copy link
Member

@fpagnoux fpagnoux left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@fpagnoux fpagnoux self-assigned this Jun 27, 2018
@claireleroy claireleroy merged commit fccbc23 into master Jun 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants