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] Passer la page de connexion à l'espace surveillant en .gjs (PIX-13332) #9469

Merged

Conversation

mcampourcy
Copy link
Contributor

🦄 Problème

Le format de composant utilisant la balise template va devenir le format par défaut d'ici la fin de l'édition Polaris de Ember.
Tous nos composants sont encore au format hbs. On peut commencer à les passer au format .gjs

🤖 Proposition

Passer les composants de la page de connexion à l'espace surveillant en gjs.

💯 Pour tester

NON REG

  • Se connecter sur Pix Certif avec [email protected], aller dans Espace surveillant
  • Entrer un numéro de session erroné
  • Vérifier que le message "La session n'existe pas ou son accès est restreint" s’affiche en cas d’erreur sur le numéro de session
  • Entrer un mot de passe erroné
  • Vérifier que le message "Le numéro de session et/ou le mot de passe saisis sont incorrects." s’affiche en cas d’erreur sur le mot de passe
  • Entrer un numéro de session finalisée
  • Vérifier que le message "La session de certification n'est plus accessible." s’affiche en cas d’erreur sur le numéro de session
  • Vérifier que le lien “Changer de compte” renvoie bien vers la page de connexion
  • Vérifier que c’est le bon email qui s’affiche en bas du formulaire (au-dessus de “Changer de compte”)
  • Vérifier que l’affichage correspond à la capture d'écran ci-jointe
  • Vérifier que la connexion pour surveiller une session est toujours OK
Capture d’écran 2024-07-08 à 17 03 28

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

@mcampourcy mcampourcy force-pushed the pix-13332-login-session-supervisor-from-hbs-to-gjs branch from 4f1e86e to 3b97445 Compare July 11, 2024 14:54
@mcampourcy mcampourcy marked this pull request as ready for review July 11, 2024 15:00
Comment on lines +15 to +17
firstName: 'Lara',
lastName: 'Pafromage',
email: 'lara.pafromage@example.com',
Copy link
Member

Choose a reason for hiding this comment

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

❤️

@mcampourcy mcampourcy requested review from a team as code owners July 12, 2024 08:59
@mcampourcy mcampourcy force-pushed the pix-13332-login-session-supervisor-from-hbs-to-gjs branch from aa65636 to 94228df Compare July 12, 2024 09:00
@frinyvonnick frinyvonnick removed request for a team July 12, 2024 12:40
@matthieu-octo
Copy link
Contributor

matthieu-octo commented Jul 17, 2024

Test fonctionnel OK ✅️

⚠️ J'ai l'impression que le deuxième commits (94228df) en venu mettre un peu de bazar et à rollback du code (création de nouveaux composants presenters).

Edit: commit fixup

@matthieu-octo matthieu-octo force-pushed the pix-13332-login-session-supervisor-from-hbs-to-gjs branch from 94228df to 50f00e7 Compare July 17, 2024 08:23
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-13332-login-session-supervisor-from-hbs-to-gjs branch from 50f00e7 to df44c25 Compare July 17, 2024 08:38
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-13332-login-session-supervisor-from-hbs-to-gjs branch from df44c25 to c147938 Compare July 17, 2024 08:46
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-13332-login-session-supervisor-from-hbs-to-gjs branch from c147938 to 77a1d40 Compare July 17, 2024 08:55
@pix-service-auto-merge pix-service-auto-merge merged commit 6cc4b46 into dev Jul 17, 2024
4 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-13332-login-session-supervisor-from-hbs-to-gjs branch July 17, 2024 09:02
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.

None yet

6 participants