Skip to content

Conversation

maxcorbeau
Copy link
Contributor

@maxcorbeau maxcorbeau commented Apr 16, 2025

🕵 Anonymiser acteurs RGPD

Carte Notion: 🕵🏻 Anonymiser acteurs RGPD

🗺️ contexte: j'avais livré une première PR RGPD (livraison initiale du 20 mars), entre temps débat juridique (décision le 31 mars), et j'ai basculé entre temps sur #1454, d'où cette PR qui se base sur la travail de fond des acteurs fermés #1454

💡 quoi: DAG de suggestions RGPD

🎯 pourquoi: anonymiser les acteurs dans notre base de données

🤔 comment: même principe que #1454:

  • dbt: modèles qui génèrent des suggestions
  • airflow: fait simplement du rafraichissement de modèle et lecture/insertion DB

🖼️ Illustration

Airflow

Même procédé que la PR de fermeture: config -> rafraichissment dbt -> suggestions

image

Django

image

✅ Reste à faire (PR en cours)

  • Ré-intégrer les modèles DBT, reformatter pour être iso avec 🚪 [ENRICH] Cloturer acteurs cessés/fermés #1454
  • Améliorer le modèle DBT pour être plus pertinent sur le matching
  • Sur la branche de base, flagger les suggestions échouées sans empêcher l'écriture
  • Tests complets sur enrich_dbt_model_to_suggestions

@maxcorbeau maxcorbeau marked this pull request as ready for review April 16, 2025 12:47
@maxcorbeau maxcorbeau requested a review from a team as a code owner April 16, 2025 12:47
@maxcorbeau maxcorbeau requested review from kolok and fabienheureux and removed request for a team April 16, 2025 12:47
@maxcorbeau maxcorbeau force-pushed the enrich_acteurs_rgpd_v1 branch from 28faaf2 to f87965d Compare April 16, 2025 13:08
@maxcorbeau maxcorbeau force-pushed the enrich_acteurs_closed_replace_v1 branch from e106fc0 to ba49c98 Compare April 21, 2025 07:39
@maxcorbeau maxcorbeau added enhancement New feature or request dbt labels Apr 21, 2025
@maxcorbeau maxcorbeau changed the title 🕵 Anonymiser acteurs RGPD 🕵 [ENRICH] Anonymiser acteurs RGPD Apr 21, 2025
@maxcorbeau maxcorbeau changed the title 🕵 [ENRICH] Anonymiser acteurs RGPD 🕵 [ENRICH] RGPD: anonymiser les acteurs Apr 21, 2025
@maxcorbeau maxcorbeau force-pushed the enrich_acteurs_rgpd_v1 branch from f87965d to 0d8b91a Compare April 21, 2025 11:56
@maxcorbeau maxcorbeau force-pushed the enrich_acteurs_closed_replace_v1 branch 2 times, most recently from 09afc1d to b233cf2 Compare April 24, 2025 05:27
@maxcorbeau maxcorbeau force-pushed the enrich_acteurs_rgpd_v1 branch from c259fbd to f734577 Compare April 24, 2025 05:30
@maxcorbeau maxcorbeau force-pushed the enrich_acteurs_closed_replace_v1 branch from dca5e56 to 947e530 Compare April 24, 2025 10:20
Base automatically changed from enrich_acteurs_closed_replace_v1 to main April 24, 2025 10:23
@maxcorbeau maxcorbeau force-pushed the enrich_acteurs_rgpd_v1 branch from f734577 to 38493e5 Compare April 24, 2025 11:22
Copy link
Contributor

@kolok kolok left a comment

Choose a reason for hiding this comment

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

Quelques relicats de la PR précédentes mais rien à dire sur le nouveau code :) (sauf le doublons qui me parait suspect, je te laisse regarder…

@maxcorbeau maxcorbeau force-pushed the enrich_acteurs_rgpd_v1 branch from b23e627 to 1eaf544 Compare April 24, 2025 14:04
@maxcorbeau maxcorbeau enabled auto-merge (squash) April 24, 2025 14:04
@maxcorbeau maxcorbeau merged commit c3e6cea into main Apr 24, 2025
8 checks passed
@maxcorbeau maxcorbeau deleted the enrich_acteurs_rgpd_v1 branch April 24, 2025 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dbt enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants