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

[TECH] ♻️ Déplacement des éléments de CertificationCourse vers le contexte src/certification/evaluation #10404

Merged
merged 3 commits into from
Nov 21, 2024

Conversation

yaf
Copy link
Member

@yaf yaf commented Oct 24, 2024

🦄 Problème

cf https://github.com/1024pix/pix/blob/dev/docs/adr/0051-nouvelle-arborescence-api.md

🤖 Proposition

Déplacement des routes de CertificationCourse vers src/certification/evaluation

🌈 Remarques

💯 Pour tester

  • Sur Pix Certif avec [email protected] (pour avoir une session V2): Créer une session de certification
    • Autoriser l'entréee session depuis l'espace surveillant
  • Depuis Pix App avec [email protected]: Rejoindre la session
    • Vérifier que le certification course a été créé (1ère route migrée : POST /api/certification-courses )
    • Rafraîchir la page pour tester la deuxième route (GET api/certification-courses/{id} )
    • Quitter la session
  • Depuis l'espace surveillant
    • Autoriser la reprise du test
  • Depuis Pix App
    • Rejoindre à nouveau la session avec le candidat
  • Constater que le certification course est correctement récupéré et que le test peut reprendre
  • Sur Pix Admin, rejoindre la session passé et accéder aux détails de la certification du candidat
  • Dans l'onglet Profil (3ème route migrée :api/admin/certifications/{id}/certified-profile)

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

@yaf yaf changed the title ♻️ refactor: move certif course to certification/evaluation co… [TECH] ♻️ Déplacement des éléments de CertificationCourse vers le contexte src/certification/evaluation Oct 24, 2024
@yaf yaf force-pushed the migrate-certification-course branch 9 times, most recently from 4642353 to 4bbaafe Compare October 24, 2024 09:14
@yaf yaf marked this pull request as ready for review October 24, 2024 09:37
@yaf yaf requested a review from a team as a code owner October 24, 2024 09:37
@HEYGUL HEYGUL force-pushed the migrate-certification-course branch from 6760137 to 5d2a278 Compare October 29, 2024 06:39
@yaf yaf force-pushed the migrate-certification-course branch 4 times, most recently from dfb0fa3 to 49801fb Compare November 7, 2024 15:46
@yaf yaf force-pushed the migrate-certification-course branch 6 times, most recently from 0b2dae6 to be1f671 Compare November 20, 2024 10:17
@yaf yaf force-pushed the migrate-certification-course branch 3 times, most recently from 2c82759 to 67dab5a Compare November 21, 2024 10:39
@P-Jeremy
Copy link
Contributor

P-Jeremy commented Nov 21, 2024

Test func création certif course OK ✅
image

Reprise de test OK ✅
image

Profil sur Pix Admin OK ✅
image

@pix-service-auto-merge pix-service-auto-merge force-pushed the migrate-certification-course branch 2 times, most recently from 4b6bb3a to e94ed28 Compare November 21, 2024 13:01
@pix-service-auto-merge pix-service-auto-merge merged commit f681713 into dev Nov 21, 2024
8 of 9 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the migrate-certification-course branch November 21, 2024 13:06
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.

6 participants