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

Déplace les paramètres des stagiaires depuis les formules vers la législation #583

Merged
merged 2 commits into from
Sep 1, 2017

Conversation

edarin
Copy link
Contributor

@edarin edarin commented Oct 14, 2016

Travail hackathon DILA "CodeGouv".

Import de paramètres depuis les simulateurs de la DILA tournant sous "G6K"

@cbenz cbenz changed the title Stage - paramètre Stagiaires - nouveaux paramètres et utilisation dans formules Oct 14, 2016
@cbenz cbenz assigned cbenz and edarin and unassigned cbenz Oct 14, 2016
@MattiSG MattiSG unassigned cbenz and edarin Mar 2, 2017
Copy link
Member

@MattiSG MattiSG left a comment

Choose a reason for hiding this comment

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

J'ai l'impression que la récupération des valeurs des taux de gratifications minimaux ont été correctement mis à jour dans les formules, mais pas ajoutés aux paramètres. ⁉️

@MattiSG MattiSG requested a review from laem March 2, 2017 15:15
@MattiSG MattiSG changed the title Stagiaires - nouveaux paramètres et utilisation dans formules Déplace les paramètres des stagiaires depuis les formules vers la législation Mar 2, 2017
@Morendil Morendil self-assigned this Aug 17, 2017
@Morendil
Copy link
Contributor

Morendil commented Aug 17, 2017

Je (re)prends.

@Morendil
Copy link
Contributor

Compte tenu de l'âge de cette PR j'ai préféré supprimer le commit mettant à jour le Changelog et le numéro de version, puis le rebase a eu pour effet de supprimer l'autre commit de @edarin - du coup ça me chiffonne que cela efface l'historique de sa contribution, mais en l'état de mes connaissances sur Git/Github je ne sais pas comment la créditer dans l'historique.

@MattiSG
Copy link
Member

MattiSG commented Aug 17, 2017

Bravo pour cette attention !

Tu peux utiliser git commit --amend --author pendant un rebase, ou un alias comme git credit :)

@laem
Copy link
Contributor

laem commented Aug 17, 2017

Merci @Morendil j'ai complètement zappé cette notification 😥

@Morendil
Copy link
Contributor

Rebase, mise à jour de la version et du Changelog, @edarin créditée dans le commit principal.

@sandcha
Copy link
Contributor

sandcha commented Sep 1, 2017

Merci @edarin @Morendil.
Vu avec @Anna-Livia et @fpagnoux. On se charge de rebaser à nouveau et de merger.

@sandcha sandcha merged commit d11d75c into master Sep 1, 2017
@sandcha sandcha deleted the stage branch September 1, 2017 16:31
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.

6 participants