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

[FEATURE] API verifiant si un utilisateur a été candidat de certif (PIX-14984) #10533

Merged
merged 3 commits into from
Nov 14, 2024

Conversation

bpetetot
Copy link
Contributor

@bpetetot bpetetot commented Nov 13, 2024

🍂 Problème

Dans le cadre d'une nouvelle fonctionnalité permettant à un utilisateur de supprimer son compte lui-même (ie. anonymiser le compte), nous souhaitons restreindre la fonctionnalité aux utilisateurs qui n'ont jamais été candidat à une certification. (car l'anonymisation des données utilisateurs de certification n'est pas effective aujourd'hui)

🌰 Proposition

Depuis un contexte "Privacy" (lié à l'anonymisation), nous allons appelé une API du contexte certification/enrolment afin d'identifier si un utilisateur (via userId) a déjà été candidat à une certification.

  • API Implementation candidates-api.js: Ajout de hasBeenCandidate vérifiant si un utilisateur a été candidat via le use-case.

  • Domain Use Case has-been-candidate.js: Ajout de hasBeenCandidate vérifiant si au moins une candidature de l'utilisateur est réconciliée (isReconciled) .

  • Repository candidate-repository.js: Ajout findByUserId récupérant toutes les candidatures et ses subscriptions pour un userId spécifique.

Exemple d'utilisation de l'api

import { hasBeenCandidate } from 'src/certification/enrolment/application/api/candidates-api.js';

await hasBeenCandidate({ userId: 123 }); // true or false

@bpetetot bpetetot self-assigned this Nov 13, 2024
@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 :

@bpetetot bpetetot force-pushed the pix-14984-check-user-has-been-candidate branch from f786c06 to 647b16c Compare November 13, 2024 11:14
Copy link
Contributor

@Steph0 Steph0 left a comment

Choose a reason for hiding this comment

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

C'est propre ! 🤩

@bpetetot bpetetot force-pushed the pix-14984-check-user-has-been-candidate branch from 647b16c to 7b84015 Compare November 14, 2024 13:30
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-14984-check-user-has-been-candidate branch 3 times, most recently from d8ba982 to 34d9bdd Compare November 14, 2024 14:33
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-14984-check-user-has-been-candidate branch from 34d9bdd to 5f9be3f Compare November 14, 2024 14:39
@pix-service-auto-merge pix-service-auto-merge merged commit 86eb8ec into dev Nov 14, 2024
9 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-14984-check-user-has-been-candidate branch November 14, 2024 14:54
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.

7 participants