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

feat: adaptation calcul anciennete requise CC rupture co #5997

Merged
merged 10 commits into from
Jul 4, 2024

Conversation

Viczei
Copy link
Contributor

@Viczei Viczei commented Jun 25, 2024

No description provided.

@Viczei Viczei linked an issue Jun 25, 2024 that may be closed by this pull request
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Copy link
Member

@maxgfr maxgfr left a comment

Choose a reason for hiding this comment

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

Nice, GG !!

Y'a juste le fichier sur lequel je t'ai mis un commentaire que je comprends pas

Comment on lines -15 to -18
case SupportedCc.IDCC2596:
return new IneligibilityLegalIndemniteLicenciement2596();
case SupportedCc.IDCC1404:
return new IneligibilityLegalIndemniteLicenciement1404();
Copy link
Member

Choose a reason for hiding this comment

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

On a pas dit qu'on se ferait ça dans une autre PR ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

En fait ici, vu qu'il n'y a pas d'ancienneté requise sur les CC de la RC. J'ai retiré ces 2 ineligibilité vu qu'elle traite de l'ancienneté requise

Copy link
Member

Choose a reason for hiding this comment

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

Oui, c'est sur la rupture co, du coup ok

@@ -26,9 +26,13 @@ contrat salarié . convention collective . cabinets médicaux . somme formule:
unité: €

# Résultat
contrat salarié . convention collective . cabinets médicaux . minimum ancienneté conventionnelle requise en année:
remplace: paramètre . minimum ancienneté conventionnelle requise en année
valeur: 1
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
valeur: 1
valeur: 1 an

@@ -64,10 +63,14 @@ contrat salarié . convention collective . batiment ouvriers employés . indemni
applicable si: age >= 55
valeur: oui

contrat salarié . convention collective . restauration rapide . minimum ancienneté conventionnelle requise en année:
remplace: paramètre . minimum ancienneté conventionnelle requise en année
valeur: 2
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
valeur: 2
valeur: 2 ans

@@ -63,10 +63,14 @@ contrat salarié . convention collective . batiment ouvriers employés bis . ind
applicable si: age >= 55
valeur: oui

contrat salarié . convention collective . batiment ouvriers employés bis . indemnité de licenciement . minimum ancienneté conventionnelle requise en année:
remplace: paramètre . minimum ancienneté conventionnelle requise en année
valeur: 2
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
valeur: 2
valeur: 2 ans

@@ -108,11 +108,15 @@ contrat salarié . convention collective . transports routiers . indemnité de l
contrat salarié . convention collective . transports routiers . indemnité de licenciement . catégorie professionnelle . Ingénieurs et cadres . résultat conventionnel . base . calcul:
valeur: 0 €

contrat salarié . convention collective . transports routiers . indemnité de licenciement . catégorie professionnelle . Ingénieurs et cadres . minimum ancienneté conventionnelle requise en année:
remplace: paramètre . minimum ancienneté conventionnelle requise en année
valeur: 3 an
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
valeur: 3 an
valeur: 3 ans

@@ -237,11 +241,15 @@ contrat salarié . convention collective . transports routiers . indemnité de l
contrat salarié . convention collective . transports routiers . indemnité de licenciement . catégorie professionnelle . TAM . résultat conventionnel . base:
valeur: 0 €

contrat salarié . convention collective . transports routiers . indemnité de licenciement . catégorie professionnelle . TAM . minimum ancienneté conventionnelle requise en année:
remplace: paramètre . minimum ancienneté conventionnelle requise en année
valeur: 2 an
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
valeur: 2 an
valeur: 2 ans

@@ -249,9 +249,13 @@ contrat salarié . convention collective . commerces de gros . somme cadre avec
unité: €

# Résultats
contrat salarié . convention collective . commerces de gros . minimum ancienneté conventionelle requise en année:
remplace: paramètre . minimum ancienneté conventionnelle requise en année
valeur: 1
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
valeur: 1
valeur: 1 an

Comment on lines -15 to -18
case SupportedCc.IDCC2596:
return new IneligibilityLegalIndemniteLicenciement2596();
case SupportedCc.IDCC1404:
return new IneligibilityLegalIndemniteLicenciement1404();
Copy link
Member

Choose a reason for hiding this comment

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

Oui, c'est sur la rupture co, du coup ok

@Viczei Viczei merged commit a20f396 into dev Jul 4, 2024
22 checks passed
@Viczei Viczei deleted the 5988-condition-anciennet-rupture-co branch July 4, 2024 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Condition ancienneté rupture co
5 participants