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] Modification des pages de CGU (PIX-15589) #11023

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from

Conversation

bpetetot
Copy link
Contributor

@bpetetot bpetetot commented Jan 8, 2025

🎄 Problème

Avec le nouveau versionnement des CGUs Pix Orga, on souhaite changer le design des pages d’acceptation et de gérer le cas des CGUs qui ont évoluées.

🎁 Proposition

Modification de la page d’acceptation des CGU:

  • Utiliser pix-orga-terms-of-service-status
    • Si accepted, se connecter à Pix Orga
    • Si requested, afficher le nouveau design de première acceptation des CGUs (voir maquette)
    • Si update-requested, afficher le nouveau design d'évolution des CGUs (voir maquette - uniquement changement de texte)

L’URL des CGUs Pix Orga en utilisant pix-orga-terms-of-service-document-path:

Versions actuelles:

🧦 Remarques

  • Suppression des CGUs Pic Orga en dur actuellement côté frontend

🎅 Pour tester

Test 1

  • Se connecter sur Pix Orga sans CGU déjà acceptées:
  • La page doit s’afficher (avec le bon lien, à tester également en .org avec une autre langue)
  • Accepter les CGUs
  • L’appel PATCH /api/users/{id}/pix-orga-terms-of-service-acceptance doit fonctionner avec l’ancien modèle des CGUs et enregistrer les CGUs

Test 2 (Nécessite de créer le legal document via le script en amont)

  • Avec le FT est activé, se connecter sur Pix Orga sans CGU déjà acceptées:
  • La page doit s’afficher (avec le bon lien, à tester également en .org avec une autre langue)
  • Accepter les CGUs
  • L’appel PATCH /api/users/{id}/pix-orga-terms-of-service-acceptance doit fonctionner avec le nouveau modèle des CGUs et enregistrer les CGUs

Test 3 (Nécessite de créer le legal document via le script en amont)

  • Créer un nouveau document avec le script suivant:
node src/legal-documents/scripts/add-new-legal-document-version.js --type 'TOS' --service 'pix-orga' --versionAt '2022-11-30'
  • Avec le FT est activé, et une nouvelle version des CGUs, se connecter sur Pix Orga avec CGU déjà acceptées dans la version précédente:
  • La page doit s’afficher avec le texte d’évolution
  • Accepter les CGUs
  • L’appel PATCH /api/users/{id}/pix-orga-terms-of-service-acceptance doit fonctionner avec le nouveau modèle des CGUs et enregistrer les CGUs

@bpetetot bpetetot self-assigned this Jan 8, 2025
@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-15589-update-acceptation-page branch 2 times, most recently from 2956478 to a3bece7 Compare January 8, 2025 16:50
@theotime2005 theotime2005 changed the title [FEATURE] Modification des pages de CGUs (PIX-15589) [FEATURE] Modification des pages de CGU (PIX-15589) Jan 9, 2025
@bpetetot bpetetot force-pushed the pix-15589-update-acceptation-page branch 9 times, most recently from 6ebfaf5 to be7a6d4 Compare January 9, 2025 17:08
@P-Jeremy P-Jeremy force-pushed the pix-15589-update-acceptation-page branch from be7a6d4 to e8adfc3 Compare January 10, 2025 09:33
@P-Jeremy
Copy link
Contributor

P-Jeremy commented Jan 10, 2025

Test func FT disabled ✅

image

Test func FT enabled + lang=en ✅
image
image

Test func FT enabled avec mise à jour des CGU ✅
image

@P-Jeremy P-Jeremy marked this pull request as ready for review January 10, 2025 10:08
@P-Jeremy P-Jeremy requested a review from a team as a code owner January 10, 2025 10:08
Copy link
Contributor

@theotime2005 theotime2005 left a comment

Choose a reason for hiding this comment

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

Revue fonctionnelle ok pour moi.

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.

4 participants