Import des EA/EATT depuis les données de l'ASP#4577
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| def sftp_host_key_fixture(): | ||
| # Use a 1024-bits key otherwise we get an OpenSSLError("digest too big for rsa key") | ||
| return ( | ||
| rsa.generate_private_key(key_size=1024, public_exponent=65537) |
Check failure
Code scanning / CodeQL
Use of weak cryptographic key
| "Entreprise Adaptée Travail Temporaire": CompanyKind.EATT, | ||
| }[v] | ||
| ) | ||
| ea_eatt_df["name"] = ea_eatt_df.name.apply(str.title) # "EA LOU JAS" becomes "Ea Lou Jas". |
There was a problem hiding this comment.
On veut vraiment faire cela ? Cela me semble plus sûr/simple de conserver le nom d'origine
There was a problem hiding this comment.
Pas convaincu non plus mais j'ai repris ce qu'on faisait avant, et l'ASP aime bien tout mettre en majuscule donc il y a plus de chance de tomber sur la bonne casse que de garder de qu'on reçois je pense, et ça doit aussi "alléger" les résultats de recherches.
Ou alors on pourrais faire un appel à l'API Sirene pour récupérer l'enseigne de l'établissement (surtout utile pour les multiétablissements), le répertoire est aussi tout en majuscule par contre. 🤔
Mais si on part sur ça je ferais une autre PR car ça serais bien de le faire aussi pour les SIAE.
There was a problem hiding this comment.
Ça me fait fortement penser à https://github.com/gip-inclusion/les-emplois/pull/3558/files
Mais effectivement vu que c'est déjà ce qu'on faisait, on peut aussi continuer (je vois qu'on le fait aussi pour les GEIQ, mais a priori pas pour les SIAE 😅 )
There was a problem hiding this comment.
Un autre "avantage" que je vois à modifier le nom lors de la création c'est qu'en rendant le truc moche ça doit pousser les gens à la corriger alors que si on laisse en tout majuscule y a moyen que ça soit jamais touché 🤷.
Sûrement une discussion à avoir avec le métier pour décider d'un truc et le faire partout une bonne fois pour toute :).
| warnings.warn( | ||
| f"Structure(s) not created because of a missing email: " | ||
| f"{info_stats['not_created_because_of_missing_email']}" | ||
| ) |
There was a problem hiding this comment.
Peut-être remplacer par des logger.warning pour avoir l'info dans Sentry (si le but est bien de vérifier avec quelqu'un que la situation est normale ?)
There was a problem hiding this comment.
J'ai supprimé car on aura sans doute jamais d'erreur, j'ai regardé 2 flux et les emails étaient tout le temps présents [1] alors que c'était jamais le cas avec l'ancien.
Par contre on pourrais reprendre la logique de l'import SIAE et enlever les lignes sans email, comme ça soit ça arrive plus tard soit la personne fera un ticket support.
[1] Je suspecte qu'il le sera toujours car ça doit être lié au compte gérant la structure dans l'extranet, donc sûrement nécessaire pour se connecter. Faudrait que je regarde dans le flux IAE si il y a des emails manquants ou pas je pense.
rsebille
force-pushed
the
rsebille/import-ea-eatt
branch
3 times, most recently
from
ede427d to
4506c1e
Compare
The remote directory is now also used by EA2 to give us their data.
Also move SFTP related fixtures to the top level conftest to match that.
Those directories are going to be used for more than employee records.
rsebille
force-pushed
the
rsebille/import-ea-eatt
branch
from
4506c1e to
95bcca7
Compare
2 participants
🤔 Pourquoi ?
Les données des EA/EATT nous était fournis par la DGEFP mais ce n'est plus le cas.
Nous aurons désormais un export hebdomadaire du flux de l'extranet EA2, celui-ci est déposé le lundi matin sur le serveur d'échange de l'ASP que nous utilisons déjà pour les fiches salariés.
🍰 Comment ?
https://github.com/gip-inclusion/itou-secrets/pull/96
import_ea_eattpour passer d'un fichier XLSX au flux EA2.import_ea_eatt.🏝️ Comment tester
Les parties SFTP sont testés unitairement mais vous pouvez récupérer les infos nécessaires depuis
itou-secrets.Pour le reste vous pouvez lancer la commande avec
--from-archiveet le flux dispo dansitou-bucket.