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

Suppression de période en dur dans la CSA #1402

Merged
merged 3 commits into from
Jan 11, 2020

Conversation

benjello
Copy link
Member

@benjello benjello commented Jan 10, 2020

  • Changement mineur.
  • Périodes concernées : toutes.
  • Zones impactées : /prelevements_obligatoires/prelevements_sociaux/taxes_salaires_main_oeuvre.
  • Détails :
    • Correction d'une formula gérant la période en dur dans le code.

Ces changements (effacez les lignes ne correspondant pas à votre cas) :

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

Quelques conseils à prendre en compte :

@benjello
Copy link
Member Author

Et merci à @elie-gerschel et @claireleroy !

@benjello benjello changed the title WIP: Contribution supplementaire apprentissage Contribution supplementaire apprentissage Jan 10, 2020
@benjello benjello requested a review from claireleroy January 10, 2020 16:22
@bonjourmauko bonjourmauko self-assigned this Jan 11, 2020
Copy link
Member

@bonjourmauko bonjourmauko left a comment

Choose a reason for hiding this comment

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

Hello et merci beaucoup pour cette contribution !!!

Serait-il possible d'introduire des tests pour celle-ci ?

Pour faciliter la tâche, j'en ai ajouté quelques uns, pourriez-vous vérifier s'ils sont corrects ?

cc @benjello @claireleroy

@benjello
Copy link
Member Author

benjello commented Jan 11, 2020

@maukoquiroga : je n'ai pas source externe de tests que de rajouter des tests comme tu l'as fait (merci en passant :-)).

@benjello
Copy link
Member Author

Je suis donc pour merger "as it is".

@bonjourmauko bonjourmauko changed the title Contribution supplementaire apprentissage Suppression de période en dur dans la Contribution Supplementaire Apprentissage Jan 11, 2020
@bonjourmauko bonjourmauko changed the title Suppression de période en dur dans la Contribution Supplementaire Apprentissage Suppression période en dur dans la Contribution Supplementaire Apprentissage Jan 11, 2020
@bonjourmauko bonjourmauko changed the title Suppression période en dur dans la Contribution Supplementaire Apprentissage Suppression de période en dur dans la CSA Jan 11, 2020
@bonjourmauko bonjourmauko force-pushed the contribution_supplementaire_apprentissage branch from a4ae76b to 0b4a437 Compare January 11, 2020 19:42
Copy link
Member

@bonjourmauko bonjourmauko left a comment

Choose a reason for hiding this comment

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

LGTM !

@bonjourmauko bonjourmauko force-pushed the contribution_supplementaire_apprentissage branch from 0b4a437 to 1bfc523 Compare January 11, 2020 19:43
@benjello benjello merged commit 11816d8 into master Jan 11, 2020
@benjello benjello deleted the contribution_supplementaire_apprentissage branch January 11, 2020 20:57
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.

3 participants