Skip to content

[FEATURE] Récupérer les CGU de l'utilisateur de Pix Certif en passant par le nouveau modèle (PIX-22441) - #17404

Open
EmmanuelleBonnemay wants to merge 6 commits into
devfrom
pix-22441-get-pix--certif-tos-status-from-new-implementation
Open

[FEATURE] Récupérer les CGU de l'utilisateur de Pix Certif en passant par le nouveau modèle (PIX-22441)#17404
EmmanuelleBonnemay wants to merge 6 commits into
devfrom
pix-22441-get-pix--certif-tos-status-from-new-implementation

Conversation

@EmmanuelleBonnemay

@EmmanuelleBonnemay EmmanuelleBonnemay commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

☀️ Problème

Afin de mettre en place le versionnement des CGU de Pix Certif, il est nécessaire de modifier la route /.../me permettant de récupérer leur valeur afin de se baser sur le nouveau modèle.

⛱️ Proposition

Sur l’appel GET /api/certif.../me, récupérer les informations de CGU en appelant la nouvelle API interne getLegalDocumentStatusByUserId du contexte legal-document

🧴 Remarques

ras

🏊 Pour tester

  • Quand le FT est activé et désactivé, (donc dans tous les cas ci-dessous) les attributs suivants doivent apparaître sur la route /api/users/me :

-- cgu
-- last-terms-of-service-validated-at
-- must-validate-terms-of-service
-- pix-app-terms-of-service-status
-- pix-app-terms-of-service-document-path

  • Quand le FT est activé :

-- Deux versions de legal-document du legal-document "Pix Certif" ont été rajoutées dans les seeds, il faut récupérer leurs versions.

[REQUESTED]-- CAS THEORIQUE : l'utilisateur n'a jamais accepté les CGU et n'est pas un élève.
Constater que la connexion avec un "non élève" (harry-cover@example.net) redirige l'utilisateur vers la page des cgu avec cgu=false, un status pix-app-terms-of-service-status = "requested", last-terms-of-service-validated-at=null, must-validate-terms-of-service=true, pix-app-terms-of-service-document-path="pix-app-tos-2022-01-01" sur la route /api/users/me

[ACCEPTED]-- Accepter les CGU. Cela rajouter une acceptance du même utilisateur sur la deuxième version des cgu (PR précédente)
Puis se déconnecter et se reconnecter avec le même utilisateur : constater que l'utilisateur est redirigé vers sa homepage, avec un cgu=true, pix-app-terms-of-service-status = "accepted", must-validate-terms-of-service = false, last-terms-of-service-validated-at=<date de validation>, pix-app-terms-of-service-document-path=pix-app-tos-2022-01-01 sur la route /api/users/me
⚠️ Question fonctionnelle : Pourquoi a-t-on besoin du document path quand les conditions sont acceptées ?

[UPDATE_REQUESTED]-- Faire un update sur les une acceptance de l'utilisateur, pour ), mais pour la première version uniquement :
UPDATE "legal-document-version-user-acceptances" SET "legalDocumentVersionId" = <id de la version précédente> WHERE "userId" = <id de l'utilisateur>;
Puis connectez-vous, constatez que l'utilisateur est redirigé vers la page d'acceptation des cgu, avec un cgu=true, pix-app-terms-of-service-status = "update-requested", must-validate-terms-of-service = true, last-terms-of-service-validated-at=null, pix-app-terms-of-service-document-path=pix-app-tos-2022-01-01 sur la route /api/users/me
=> last-terms-of-service-validated-at est ramené à null dans ce cas car dans le modèle UserWithActivity, la valeur de acceptedAt (qui est nulle quand le statut est UPDATE_REQUESTED) est reportée sur le last-terms-of-service-validated-at

[NOT_APPLICABLE]-- Constater que la connexion avec un élève (bob.leponge.0202) ne dirige pas l'utilisateur vers la page des cgu mais vers sa homepage, et que pix-app-terms-of-service-status = "not-applicable", avec cgu=false, must-validate-terms-of-service = false, last-terms-of-service-validated-at=null, pix-app-terms-of-service-document-path=null

@pix-bot-github

pix-bot-github commented Sep 4, 2026

Copy link
Copy Markdown

@widy-martial
widy-martial requested review from a team as code owners September 4, 2026 15:26
@EmmanuelleBonnemay
EmmanuelleBonnemay force-pushed the pix-22441-get-pix--certif-tos-status-from-new-implementation branch from 9843676 to ce45934 Compare September 7, 2026 08:09
@Libouk
Libouk force-pushed the pix-22441-get-pix--certif-tos-status-from-new-implementation branch from 8c71f02 to 0cfd9b5 Compare September 7, 2026 09:45
@nlepage
nlepage force-pushed the pix-22441-get-pix--certif-tos-status-from-new-implementation branch from 65c1f11 to 9877f5a Compare September 7, 2026 14:03
@EmmanuelleBonnemay
EmmanuelleBonnemay force-pushed the pix-22441-get-pix--certif-tos-status-from-new-implementation branch from 9877f5a to dbcf4e7 Compare September 7, 2026 15:38
@EmmanuelleBonnemay
EmmanuelleBonnemay force-pushed the pix-22441-get-pix--certif-tos-status-from-new-implementation branch from dbcf4e7 to 1152437 Compare September 7, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants