Skip to content

Conversation

@xav-car
Copy link
Contributor

@xav-car xav-car commented Jun 12, 2025

🔆 Problème

Lorsqu'on fait un import sur une orga SUP pour remplacer des prescrits, on supprime les anciens prescrits non présents dans le fichier avant d'ajouter / modifier ceux du fichier. mais les prescrits ne sont jamais anonymisés.

⛱️ Proposition

Modifier le job d'import du SUP pour qu'il utilise le usecase de suppression des prescrits car il supporte le feature flag (et on ré-utilise du code et c'est beau)

🌊 Remarques

En passant par le usecase de suppression de learner, on modifie aussi le updatedAt sur l'organizationLearner supprimé (ce qui n'était pas fait avant)

🏄 Pour tester

Avec le Flag d'anonymisation à false

  • Faire un import sup avec le fichier sup-ok (voir les fichiers de test de la prescription)
  • Enlever 2 prescrit du fichier
  • Faire un import sup avec le nouveau fichier
  • Se connecter avec pgsql-console
select * from "organization-learners" where "organizationId"=1001
  • valider que les learners sont supprimés mais pas anonymisés
select * from "campaign-participations" JOIN "organization-learners" ON "organization-learners".id="organizationLearnerId" AND "organization-learners"."deletedAt" IS NOT NULL  where "organizationId"=1001;
  • valider que les participations sont supprimées mais pas anonymisées.

Avec le Flag d'anonymisation à true

  • Dans Faire un import sup avec le fichier sup-ok (voir les fichiers de test de la prescription)
  • Faire un import sup avec le fichier modifié
  • Se connecter avec pgsql-console
select * from "organization-learners" where "organizationId"=1001
  • valider que les learners sont supprimés ET anonymisés.
select * from "campaign-participations" JOIN "organization-learners" ON "organization-learners".id="organizationLearnerId" AND "organization-learners"."deletedAt" IS NOT NULL  where "organizationId"=1001;
  • valider que les participations sont supprimées ET anonymisées.

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@xav-car xav-car force-pushed the pix-18074/add-campaign-participation-deletion-on-learner-import-sup branch from 8c3b31a to 2df5a4d Compare June 12, 2025 13:20
@lionelB lionelB force-pushed the pix-18074/add-campaign-participation-deletion-on-learner-import-sup branch from 2df5a4d to 598bfc4 Compare June 12, 2025 14:12
@xav-car xav-car marked this pull request as ready for review June 13, 2025 08:08
@xav-car xav-car requested a review from a team as a code owner June 13, 2025 08:08
@xav-car xav-car force-pushed the pix-18074/add-campaign-participation-deletion-on-learner-import-sup branch from ada135b to 2eddd07 Compare June 13, 2025 08:08
@VincentHardouin VincentHardouin added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed Need PO validation for this functionally labels Jun 13, 2025
@lionelB lionelB force-pushed the pix-18074/add-campaign-participation-deletion-on-learner-import-sup branch 2 times, most recently from 4ae8906 to de29b1e Compare June 16, 2025 13:34
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-18074/add-campaign-participation-deletion-on-learner-import-sup branch from de29b1e to c4af4fc Compare June 16, 2025 13:44
@pix-service-auto-merge pix-service-auto-merge merged commit ac52e42 into dev Jun 16, 2025
10 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-18074/add-campaign-participation-deletion-on-learner-import-sup branch June 16, 2025 13:51
pix-service pushed a commit that referenced this pull request Jun 17, 2025
# [5.136.0](v5.135.0...v5.136.0) (2025-06-17)

### 🚀 Amélioration

- [#12514](#12514) Ajouter la suppression de participations / assessment / badge / trainings lors de la suppression d'un import SUP (PIX-18074)
- [#12513](#12513) Faire disparaître les élements du toaster automatiquement lors du changement de page sur Pix Admin (PIX-18169).
- [#12387](#12387) Nouveaux messages d'erreur lors d'une tentative de connexion (PIX-17949)
- [#12554](#12554) Support des web-components Vue >=3.5.15 (PIX-18219)

### 🐛 Correction

- [#12547](#12547) Éviter d'appeller le LLM en mode preview (PIX-18295)

### 🏗️ Tech

- [#12555](#12555) Suppression d'un script inutilisé.
- [#12542](#12542) Suppression du service Campaign Media Compliance (PIX-18206).

### ⬆️ Montée de version

- [#12551](#12551) Update dependency @1024pix/epreuves-components to ^0.4.0 (junior)
- [#12552](#12552) Update dependency @1024pix/epreuves-components to ^0.4.0 (mon-pix)
- [#12553](#12553) Update dependency ember-source to ~6.5.0 (junior)
- [#12539](#12539) Update slackapi/slack-github-action action to v2.1.0 (workflows)
@pix-service
Copy link
Contributor

🎉 This PR is included in version 5.136.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants