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] Ajout du nouveau formulaire de connexion Pix App (PIX-14002) #10184

Merged
merged 7 commits into from
Sep 30, 2024

Conversation

bpetetot
Copy link
Contributor

@bpetetot bpetetot commented Sep 25, 2024

🦄 Problème

Dans le cadre du nouveau design des pages d'authentification de Pix App, nous modifions le design du formulaire de login.

🤖 Proposition

Cette PR contient uniquement la migration du design du formulaire de signin (email/password) de la page d'authentification:

  • Nouveau design du formulaire
  • Brancher le bouton "Je me connecte" (Authentification)
  • Brancher le lien "Mot de passe oublié"
  • Gérer des erreurs
  • Ajouter les tests pour la nouvelle version (sous feature switch)
    De plus, les premiers feedbacks de l'équipe design ont été pris en compte dans cette PR.

Note

Le test d'acceptance suivant a été renommé et déplacé:

  • Avant: mon-pix/tests/acceptance/authentication-test.js
  • Après: mon-pix/tests/acceptance/authentication/authentication-redirections-test.js

💯 Pour tester

  • Dans le fichier .env de l'api (déjà fait pour la RA), ajouter:
FT_NEW_AUTHENTICATION_DESIGN_ENABLED=true
  • Aller sur Pix App et vérifier que la nouvelle mire de connexion s'affiche correctement (formulaire de connexion uniquement)

Tester:

  • Une connexion "classique" => superadmin@example.net
    • Les champs doivent être obligatoires.
    • Le bouton "Je ne connecte" est désactivé tant que les 2 champs ne sont pas remplis.
    • L'utilisateur est redirigé sur pix app une fois connecté.
  • Le lien "Mot de passe oublié".
    • Redirige vers la page "Mot de passe oublié" (ancien design)
  • Les cas d'erreurs:
    • Utilisateur avec mot de passe temporaire => hermione@school.net / xzjgcg56
    • Utilisateur temporairement bloqué: temporary-blocked@example.net (faire quelques tentatives avec un mauvais mot de passe pour que ça bloque)
    • Utilisateur bloqué: blocked@example.net (faire des 1 ou 2 tentatives avec un mauvais mot de passe pour que ça bloque)
  • [Sur la version pix.org] Le changement de langue:
    • Le texte de la page doit changer en fonction de la langue choisie
    • Un message d'erreur affiché doit également voir son texte changer (ancien bug corrigé)

Screenshot

image

@bpetetot bpetetot self-assigned this Sep 25, 2024
@bpetetot bpetetot changed the base branch from dev to pix-13996-prepare-layout-template September 25, 2024 14:19
@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 :

@bpetetot bpetetot force-pushed the pix-14002-new-signin-form branch 3 times, most recently from bbd4b72 to 3ad4802 Compare September 25, 2024 14:35
@bpetetot bpetetot force-pushed the pix-14002-new-signin-form branch 2 times, most recently from a6ccf4a to d4fd0aa Compare September 25, 2024 15:39
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-13996-prepare-layout-template branch 2 times, most recently from 0335ae4 to fee1e58 Compare September 26, 2024 08:30
Base automatically changed from pix-13996-prepare-layout-template to dev September 26, 2024 08:35
@bpetetot bpetetot force-pushed the pix-14002-new-signin-form branch 2 times, most recently from 740cf49 to f389b80 Compare September 26, 2024 13:42
@bpetetot bpetetot marked this pull request as ready for review September 26, 2024 14:25
@bpetetot bpetetot requested a review from a team September 26, 2024 14:48
@mariannebost
Copy link
Contributor

Les cas d'erreurs pour
Utilisateur temporairement bloqué: temporary-blocked@example.net
Utilisateur bloqué: blocked@example.net
Ne semble pas fonctionner car j'arrive à me connecter

@bpetetot
Copy link
Contributor Author

bpetetot commented Sep 27, 2024

Les cas d'erreurs pour
Utilisateur temporairement bloqué: temporary-blocked@example.net
Utilisateur bloqué: blocked@example.net
Ne semble pas fonctionner car j'arrive à me connecter

C'était un souci de process (il fallait faire des 1 ou 2 tentatives avec un faux mot de passe pour que ça bloque), normalement c'est bon maintenant ;)

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

Rien de plus à ajouter à la super revue réalisée par @er-lim

Copy link
Contributor

@er-lim er-lim left a comment

Choose a reason for hiding this comment

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

Tests en RA sur firefox Desktop + tablette ✅

@pix-service-auto-merge pix-service-auto-merge merged commit 65acc35 into dev Sep 30, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-14002-new-signin-form branch September 30, 2024 15:18
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