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] Enregistrer la confirmation de lecture des écrans d'instructions sur Pix App (PIX-12907). #9443

Conversation

AndreiaPena
Copy link
Member

@AndreiaPena AndreiaPena commented Jul 3, 2024

🦄 Problème

Nous avons créé la route API permettant de confirmer que le candidat à pris connaissances des écrans d'instructions V3.
Il manque la partie front pour rendre l'enregistrement opérationnel.

🤖 Proposition

Implémenter l'appel à la route coté Pix App

💯 Pour tester

  • Se connecter sur Pix App avec certif-success@example.net
  • Récupérer les infos candidat d'une session V3
  • Remplir ces infos dans le formulaire d'entrée en certification
  • Passer tous les écrans, valider la checkbox et continuer
  • Constater que l'on arrive au code candidat
  • En BDD constater que le booléen dans la table certification-candidate est passé à true pour le candidat
  • Revenir au formulaire d'entrée en certif et re-remplir les infos
  • Constater que l'on ne passe plus par les écrans et direct au code candidat

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

@P-Jeremy P-Jeremy force-pushed the pix-12907-save-certification-instructions-confirmation-front-part branch 2 times, most recently from aea4011 to 3bfeebe Compare July 3, 2024 13:34
@AndreiaPena AndreiaPena force-pushed the pix-12907-save-certification-instructions-confirmation-front-part branch 4 times, most recently from a7b5f61 to f75dd8c Compare July 3, 2024 14:26
@P-Jeremy P-Jeremy force-pushed the pix-12907-save-certification-instructions-confirmation-front-part branch 3 times, most recently from 7e1f58d to f9cfb4c Compare July 4, 2024 08:29
@AndreiaPena AndreiaPena force-pushed the pix-12907-save-certification-instructions-confirmation-front-part branch from f9cfb4c to 6434b42 Compare July 4, 2024 09:26
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-12901-save-certification-instructions-for-candidate branch 5 times, most recently from 101b087 to c10abd5 Compare July 4, 2024 13:08
Base automatically changed from pix-12901-save-certification-instructions-for-candidate to dev July 4, 2024 13:14
@P-Jeremy P-Jeremy force-pushed the pix-12907-save-certification-instructions-confirmation-front-part branch 2 times, most recently from 95e3f03 to 7f134bb Compare July 4, 2024 14:19
@AndreiaPena AndreiaPena marked this pull request as ready for review July 4, 2024 14:50
@AndreiaPena AndreiaPena requested a review from a team as a code owner July 4, 2024 14:50
@P-Jeremy P-Jeremy force-pushed the pix-12907-save-certification-instructions-confirmation-front-part branch from 7f134bb to 03b451a Compare July 4, 2024 14:50
@alexandrecoin
Copy link
Contributor

Fonctionnel OK

@alexandrecoin alexandrecoin added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed labels Jul 5, 2024
@AndreiaPena AndreiaPena force-pushed the pix-12907-save-certification-instructions-confirmation-front-part branch from 03b451a to a4e71fb Compare July 8, 2024 08:42
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-12907-save-certification-instructions-confirmation-front-part branch from a4e71fb to d4d0f34 Compare July 8, 2024 09:20
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-12907-save-certification-instructions-confirmation-front-part branch from d4d0f34 to 7987db4 Compare July 8, 2024 09:20
@pix-service-auto-merge pix-service-auto-merge merged commit 09280bc into dev Jul 8, 2024
6 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-12907-save-certification-instructions-confirmation-front-part branch July 8, 2024 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants