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

[BUGFIX] Corriger l'inscription en local sur la double mire SCO (PIX-15290) #10537

Merged
merged 2 commits into from
Nov 15, 2024

Conversation

er-lim
Copy link
Contributor

@er-lim er-lim commented Nov 13, 2024

🍂 Problème

Lorsqu'on veut s'inscrire, en local, sur la double mire SCO, la route /api/sco-organization-learners/possibilities est appelée avec succès mais la suite du formulaire ne s'affiche pas.

C'est causé par l'erreur Ember Data suivante :

Error: Expected an id [CLIENT_ORIGINATED] sco-organization-learner:null (@lid:sco-organization-learner-SCOBADGE1_Potter) in response {"id":null,"type":"sco-organization-learner","attributes":{"birthdate":"2012-12-12","firstName":"Harry","lastName":"Potter","username":"harry.potter1212"},"relationships":{}}

🌰 Proposition

Ajouter, dans le serializer correspondant, un id avec comme valeur le username qui est unique pour ce cas.

🎃 Remarques

  • On a aussi mis une vérification sur l'objet errorResponse.errors dans la gestion des erreurs pour éviter une autre erreur lorsque que le champ errors est undefined.
  • Une correction a aussi été effectuée sur le serializer serializeExternal pour éviter d'utiliser la fonction randomUUID mais plutôt l'access token comme id 😄

🪵 Pour tester (en local)

  • Rejoindre une campagne SCO (ex: SCOBADGE1) sur la route /campagnes/SCOBADGE1/presentation.
  • Cliquer sur le bouton Je commence.
  • Remplir les informations du formulaire Je m'inscris sur Pix avec un compte n'ayant pas de méthodes de connexion (ex: Harry Potter 12/12/2012).
  • Cliquer sur le bouton Je m'inscris.
  • Vérifier que la suite du formulaire s'affiche bien 🎉

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

Copy link
Contributor

@lego-technix lego-technix left a comment

Choose a reason for hiding this comment

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

✅ Lu et testé fonctionnellement avec succès avec Firefox

@er-lim er-lim force-pushed the pix-15290-fix-sco-authentication-pages branch 2 times, most recently from 8842654 to ea40ef0 Compare November 14, 2024 14:33
er-lim and others added 2 commits November 15, 2024 07:56
and token as id for serializeExternal method

Co-authored-by: Emmanuelle Bonnemay <[email protected]>
when errors is not thrown from the API

Co-authored-by: Emmanuelle Bonnemay <[email protected]>
@pix-service-auto-merge pix-service-auto-merge merged commit 5068b71 into dev Nov 15, 2024
8 of 9 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15290-fix-sco-authentication-pages branch November 15, 2024 08:03
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