Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TECH] ajoute le campaignParticipationId dans la table knowledge-element-snapshots (PIX-15755) #11103

Merged

Conversation

lionelB
Copy link
Member

@lionelB lionelB commented Jan 13, 2025

❄️ Problème

Dans le cadre de la mise en conformité RGPD, on a rajouté une colonne campaignParticipationId pour lier les knowledge-element-snapshots à une participation plutôt qu'a un utilisateur. Pour l'instant une bonne partie des enregistrement de la table knowledge-element-snapshot est vide

⛄ Proposition

On rempli la colonne

🍦Remarques

RAS

⛸️ Pour tester

Participations liées

  • Se connecter sur pg et vider la column campaignParticipationId de la table knowlege-element-snapshots
  • Ouvrir un shell sur la RA
  • Lancer le script node scripts/prod/populate-campaign-participation-id.js
  • Ouvrir pg-console
  • voir que la column campaignParticipationId de la table knowlege-element-snapshots est renseignée

Participations anonymisées

  • Se connecter sur pg et vider la column campaignParticipationId de la table knowlege-element-snapshots
  • Ouvrir un shell sur la RA
  • Lancer le script node scripts/prod/delete-and-anonymise-organization-learners.js --organizationLearnerIds 123
  • Lancer le script node scripts/prod/populate-campaign-participation-id-in-knowledge-element-snapshot.js
  • Se connecter sur pg et voir que la column campaignParticipationId de la table knowlege-element-snapshots est renseigner

@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 :

@lionelB lionelB force-pushed the pix-15755/populate-kes-campaign-participation-id branch 3 times, most recently from d27b555 to bacf2fc Compare January 15, 2025 16:17
@lionelB lionelB force-pushed the pix-15755/populate-kes-campaign-participation-id branch from bacf2fc to d04aed5 Compare January 15, 2025 18:27
@Alexandre-Monney Alexandre-Monney changed the title [TECh] ajoute le campaignParticipationId dans la table knowledge-element-snapshots [TECH] ajoute le campaignParticipationId dans la table knowledge-element-snapshots (PIX-15755) Jan 16, 2025
@alicegoarnisson alicegoarnisson marked this pull request as ready for review January 16, 2025 13:19
@alicegoarnisson
Copy link
Contributor

Func ok, vu en mob 🦕 🐈‍⬛ 🦦

@lionelB lionelB force-pushed the pix-15755/populate-kes-campaign-participation-id branch from d04aed5 to 8e3288b Compare January 17, 2025 09:00
Copy link
Contributor

@xav-car xav-car left a comment

Choose a reason for hiding this comment

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

Re Func OK en mob 🦦 🐕 🦕 🐈‍⬛ 🐼

Copy link
Contributor

@xav-car xav-car left a comment

Choose a reason for hiding this comment

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

Tech OK pareil en mob moyennant les mini retours

@lionelB lionelB force-pushed the pix-15755/populate-kes-campaign-participation-id branch from 8e3288b to a609cea Compare January 17, 2025 16:01
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15755/populate-kes-campaign-participation-id branch from a609cea to c0d7da8 Compare January 17, 2025 20:11
@pix-service-auto-merge pix-service-auto-merge merged commit 42808a6 into dev Jan 17, 2025
8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15755/populate-kes-campaign-participation-id branch January 17, 2025 20:18
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.

6 participants