Pilotage: Étendre commande extract_c2_users avec plus des données #5498
Conversation
calummackervoy
added
the
no-changelog
calummackervoy
force-pushed
the
calum/export-siae
branch
from
e0a76b1 to
a10c768
Compare
calummackervoy
changed the title
calummackervoy
force-pushed
the
calum/export-siae
branch
from
a10c768 to
9e735a8
Compare
|
@notion-bot non, pas du tout 😄 |
| if include_siae: | ||
| siae_csv_rows = [] | ||
|
|
||
| siae_memberships = CompanyMembership.objects.select_related("user", "company").filter(is_active=True) |
There was a problem hiding this comment.
Vu que tu ne fais pas de filtres ici tu ne récupères pas que les SIAE mais toutes les entreprises ;), donc soit ajouter le filtre soit changer la mention "siae"
There was a problem hiding this comment.
Ah bon, j'étais sous l'impression qu'elles sont tous SIAEs 🤦 quelles types d'enterprises sont les SIAEs ?
class CompanyKind(models.TextChoices):
ACI = "ACI", "Atelier chantier d'insertion"
AI = "AI", "Association intermédiaire"
EA = "EA", "Entreprise adaptée"
EATT = "EATT", "Entreprise adaptée de travail temporaire"
EI = "EI", "Entreprise d'insertion" # Regroupées au sein de la fédération des entreprises d'insertion.
EITI = "EITI", "Entreprise d'insertion par le travail indépendant"
ETTI = "ETTI", "Entreprise de travail temporaire d'insertion"
GEIQ = "GEIQ", "Groupement d'employeurs pour l'insertion et la qualification"
OPCS = "OPCS", "Organisation porteuse de la clause sociale"Ça vaut une méthode sur CompanyKind tu penses pour récupérer une liste des types SIAE ? Ça va faciliter la maintenance
There was a problem hiding this comment.
Les types pour les SIAEs sont ici :
les-emplois/itou/companies/enums.py
Lines 21 to 27 in 239c167
Les autres types ne font pas des parcours IAE mais reste tout de même dans la famille de l'inclusion, historiquement le modèle Company était nommé Siae mais avec l'ajout des GEIQ on a préféré renommer pour éviter des incompréhensions entre la partie technique et le métier.
Pas spécialement convaincu du besoin de la méthode vu qu'on a la liste mais à réfléchir
There was a problem hiding this comment.
historiquement le modèle Company était nommé Siae
OK j'avais vu ça mais j'avais raté la motivation
Le nom du variable suggère qu'il y a au moins la possibilité d'avoir une SIAE sans convention ? Sauf que c'est un mal compris j'ajouterais un autre valeur pour tous les SIAEs même si c'est juste un return SIAE_WITH_CONVENTION_KINDS ?
There was a problem hiding this comment.
Le nom du variable suggère qu'il y a au moins la possibilité d'avoir une SIAE sans convention ?
Le nom le suggère mais ce n'est pas le cas, et ça ne devrais jamais être, car on se base sur les données officielles de l'ASP.
L'incompréhension c'est du legacy, ça vient du changement de nom du modèle et que la variable n'a pas été renommée en quelque chose de plus compréhensible à l'époque.
There was a problem hiding this comment.
Merci pour les explications ! Je vais utiliser SIAE_WITH_CONVENTION_KINDS dans ce PR et ouvrir un autre pour le renommage
Adds two new optional parameters and additional data in the generated CSV files, now divided between institutions and SIAEs.
calummackervoy
force-pushed
the
calum/export-siae
branch
from
9e735a8 to
bbfc031
Compare
🤔 Pourquoi ?
Carte Notion : https://www.notion.so/gip-inclusion/Script-Extraites-Utilisateurs-1895f321b60480eb8d5accbfb4d28e1f
La modification d'un script pour faciliter les devs à exporter des CSVs des données utilisateurs pour le métier.
🍰 Comment ?
J'ai modifié la commande d'export existant pour ajouter des nouveaux données. J'ai inclus deux paramètres optionnels parce que les données démandé par le métier peuvent varier.
C'était discuté, mais je n'ai pas ciblé la possibilité des filtres sur les données (par example par département ou par date) pour minimiser les changements de ce PR dans la contexte qu'ils font pas partie des besoins du moment, et c'est possible que ça ne va pas arriver pendant les prochains 6 mois.
🚨 À vérifier
🏝️ Comment tester ?
python manage.py extract_c2_users --include-siae --include-prescribers, et les variations sans ces paramètresexports/