Skip to content

Fiches salarié : Récolter et transmettre les informations spécifiques aux EITI [GEN-2125]#5197

Merged
rsebille merged 3 commits intomasterfrom
rsebille/er-new-fields-for-eiti
Dec 11, 2024
Merged

Fiches salarié : Récolter et transmettre les informations spécifiques aux EITI [GEN-2125]#5197
rsebille merged 3 commits intomasterfrom
rsebille/er-new-fields-for-eiti

Conversation

@rsebille
Copy link
Contributor

@rsebille rsebille commented Dec 3, 2024

🤔 Pourquoi ?

Nouvelle version du cahier des charges (v1.07)

🏝️ Comment tester

Pour l'interface :

  1. Se connecter avec une EITI
  2. Utiliser le parcours de création classique d'une FS, les nouveaux champs seront proposés

Pour tester l'envoi :

  1. Créer une recette jetable et lui lier c1-ssh-fs-config
  2. Créer et compléter des FS avec les cas que l'ont souhaite transmettre, le SIRET de la structure utilisé n'est pas important seule la mesure l'est car on le réécris en mode "test".
  3. Lancer les scripts transfer_employee_records avec --upload

Pour tester l'intégration et/ou les erreurs :

  1. Il faut contacter l'équipe RIAE une fois l'envoi fait afin qu'il lance leur moulinette
  2. Lancer les scripts transfer_employee_records avec --download

💻 Captures d'écran

Champs à l'étape 3 :
image
Champs dans l'admin :
image

@rsebille rsebille added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC ajouté Ajouté dans le changelog. labels Dec 3, 2024
@rsebille rsebille self-assigned this Dec 3, 2024
@rsebille
Copy link
Contributor Author

rsebille commented Dec 3, 2024

@xavfernandez Si tu veux jeter un oeil.
Je fait les tests d'envoi et d'intégration dans l'après-midi donc possible qu'il y ai des ajustements au niveau de certain des nouveaux champs, mais sinon ça devrais plus trop bouger.
Les tests pour l'UI arriveront plus tard mais ils sont bien prévu :).

@github-actions
Copy link

github-actions bot commented Dec 3, 2024

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@rsebille rsebille force-pushed the rsebille/er-new-fields-for-eiti branch 2 times, most recently from a90944b to 15b7e3f Compare December 4, 2024 10:38
@rsebille rsebille force-pushed the rsebille/er-new-fields-for-eiti branch 2 times, most recently from d95054b to 16e57ca Compare December 9, 2024 16:06
@rsebille rsebille requested a review from xavfernandez December 9, 2024 17:28
@rsebille rsebille marked this pull request as ready for review December 9, 2024 17:28
<div class="d-flex flex-row justify-content-between align-items-center">
<p class="mb-0 text-start">
<i class="ri-information-line fw-medium pe-2" aria-hidden="true"></i>
<strong>L'envoi et le traitement des fiches salarié sont suspendus en attendant la fin de la maintenance de l'Extranet IAE 2.0 de l'ASP prévue le 12 décembre.</strong>
Copy link
Contributor

Choose a reason for hiding this comment

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

On met le message mais le cron n'est pas désactivé ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Mon cerveau c'est allumé hier nuit, tester le drapeau sert à rien car il n'est actif que sur c1-worker donc je vire et on fera un revert

]
help_texts = {
"cape_freelance": "Contrat d'Appui au Projet Entreprise",
"cesa_freelance": "Contrat Entrepreneur Salarié. Associé",
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
"cesa_freelance": "Contrat Entrepreneur Salarié. Associé",
"cesa_freelance": "Contrat Entrepreneur Salarié Associé",

@rsebille rsebille force-pushed the rsebille/er-new-fields-for-eiti branch from ba6ed8d to 178f17f Compare December 10, 2024 19:20
Copy link
Contributor

@xavfernandez xavfernandez left a comment

Choose a reason for hiding this comment

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

J'aurais beaucoup à dire sur tout ce qu'on envoie mais bon, c'est la spec 🤷‍♂️ .

@rsebille rsebille force-pushed the rsebille/er-new-fields-for-eiti branch 3 times, most recently from e310105 to 03d5997 Compare December 11, 2024 11:36
@rsebille
Copy link
Contributor Author

Dernière version, avec tous les retours traités et des tests pour l'étape 3 et 5 des EITI.

@rsebille rsebille force-pushed the rsebille/er-new-fields-for-eiti branch from 03d5997 to 6ed08da Compare December 11, 2024 13:32
"""
Return the denomination of a worker for the company, i.e "salarié" or "travailleur indépendant".
"""
return "travailleur indépendant" if company.kind == CompanyKind.EITI else "salarié"
Copy link
Contributor

Choose a reason for hiding this comment

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

Il va falloir en changer des templates 😿
On continue à parler de Fiche salarié tout de même ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oui, ça va bouger des trucs partout mais c'est pas la priorité non plus donc ça sera sûrement fait en mode boost petit à petit.
Pour les TI l'ASP ne parle pas de salarié, donc ça changera peut-être, à voir 🤷.

Message(level=25, message="La création de cette fiche salariée est terminée||Vous pouvez suivre l'avancement de son traitement par l'ASP en sélectionnant les différents statuts.", extra_tags='toast')
# ---
# name: TestCreateEmployeeRecordStep5ForEITI.test_employee_record_status
Message(level=25, message="La création de cette fiche salariée est terminée||Vous pouvez suivre l'avancement de son traitement par l'ASP en sélectionnant les différents statuts.", extra_tags='toast')
Copy link
Contributor

Choose a reason for hiding this comment

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

Pas lié à cette PR mais

Suggested change
Message(level=25, message="La création de cette fiche salariée est terminée||Vous pouvez suivre l'avancement de son traitement par l'ASP en sélectionnant les différents statuts.", extra_tags='toast')
Message(level=25, message="La création de cette fiche salarié est terminée||Vous pouvez suivre l'avancement de son traitement par l'ASP en sélectionnant les différents statuts.", extra_tags='toast')

a priori ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Bizarre que ça n'ai pas été corrigé lors de la passe globale, la ligne est assez vieille 🤔.

@rsebille rsebille added this pull request to the merge queue Dec 11, 2024
Merged via the queue into master with commit 6d7c304 Dec 11, 2024
@rsebille rsebille deleted the rsebille/er-new-fields-for-eiti branch December 11, 2024 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC ajouté Ajouté dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants