Skip to content

feat(onboarding): remplace la signature de charte par la synthese des engagements - #1328

Merged
cesar-richard merged 4 commits into
frigofrom
feat/3429-refonte-engagements
Aug 13, 2026
Merged

feat(onboarding): remplace la signature de charte par la synthese des engagements#1328
cesar-richard merged 4 commits into
frigofrom
feat/3429-refonte-engagements

Conversation

@cesar-richard

Copy link
Copy Markdown
Collaborator

Objectif

Fixes https://gitlab.data.aphp.fr/ID/design-et-produit/ipa/cohort360/gestion-de-projet/-/work_items/3429

Mise en place de la structure de l'étape 2 refondue : tag d'engagement, callout d'exemple et validation par synthèse.

Changements réalisés

  • commitments.ts : libellés des 10 engagements et calcul du tag « Engagement N ».
  • ScreenTag et champ tag sur la configuration d'écran, rendu au-dessus du contenu.
  • InfoCallout : encart d'information réutilisable, appliqué à l'écran de l'EDS.
  • CommitmentsSummary : récapitulatif des 10 engagements, case de certification et bouton « Valider » qui enregistre la validation via signCharter().
  • Suppression de l'écran de signature de la charte PDF.

Impacts

Front. La validation de l'étape 2 appelle toujours POST /users/me/onboarding/charter/, la traçabilité en base est inchangée.

Tests réalisés

Tests unitaires ajoutés et mis à jour.

Risques

Les 10 écrans d'engagement arrivent dans un second temps : la synthèse liste des engagements que le parcours ne présente pas encore. Le lien de téléchargement du récapitulatif (RG3429.06) reste à faire, le document n'étant pas disponible.

@cesar-richard cesar-richard self-assigned this Aug 13, 2026
@sonarqubecloud

Copy link
Copy Markdown

@cesar-richard
cesar-richard merged commit 4cd6830 into frigo Aug 13, 2026
10 checks passed
@cesar-richard
cesar-richard deleted the feat/3429-refonte-engagements branch August 13, 2026 18:59
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.

1 participant