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] Utiliser PixCode dans la vérification des certificats sur Pix App (PIX-16429). #11343

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

AndreiaPena
Copy link
Member

@AndreiaPena AndreiaPena commented Feb 5, 2025

🥞 Problème

Actuellement les champs code sur Pix App ne sont pas des composants Pix UI

🥓 Proposition

Utiliser le nouveau composant PixCode.

🧃 Remarques

Création d'un composant pour cette page

😋 Pour tester

Aller sur /verification-certificat et faire une non régression :

  • ne pas mettre de code et valider => erreur sous le champ
  • mettre un code sans le "P-" => erreur sous le champ
  • mettre un code avec "P-" mais inconnu de la BDD => erreur en notificationAlert
  • couper la connexion et valider => erreur par défaut en notificationAlert
  • CAS PASSANT : P-12836133 => vérifier qu'on est redirigé vers le résultat
Capture d’écran 2025-02-05 à 14 53 18

@AndreiaPena AndreiaPena added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally team-certif cross-team Toutes les équipes de dev labels Feb 5, 2025
@AndreiaPena AndreiaPena self-assigned this Feb 5, 2025
@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 :

@AndreiaPena AndreiaPena force-pushed the pix-16429-use-pix-code-on-attestations-verification branch from d31ef10 to 7486582 Compare February 5, 2025 14:43

@tracked certificateVerificationCode = null;
@tracked errorMessage = null;
@tracked status = 'default';
Copy link
Member Author

Choose a reason for hiding this comment

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

A gérer coté PixUI : le liseré rouge en cas d'erreur n'apparaît pas.


@action
clearErrors() {
this.errorMessage = null;
Copy link
Member

Choose a reason for hiding this comment

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

suggestion lorsque le champ a le focus et qu'on valide avec "entrée" le message d'erreur apparait brievement et disparait. on pourrait rajouter un check sur le clearErrors(event) pour ne rien faire si l'event reçu correspond à la touche entrée

Copy link
Member

Choose a reason for hiding this comment

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

au alors rajouter event.stopImmediatePropagation() dans checkCertificate

Copy link
Member

Choose a reason for hiding this comment

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

oups, je viens de vérifier, le problème était la avant

Copy link
Member

@lionelB lionelB left a comment

Choose a reason for hiding this comment

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

Top ! quelques commentaires, mais rien de bloquant

@AndreiaPena AndreiaPena force-pushed the pix-16429-use-pix-code-on-attestations-verification branch from 7486582 to 0a4a25e Compare February 6, 2025 09:32
@AndreiaPena AndreiaPena force-pushed the pix-16429-use-pix-code-on-attestations-verification branch from 0a4a25e to 0e9a76c Compare February 6, 2025 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev Design Review OK 👀 Func Review Needed Need PO validation for this functionally 👀 Tech Review Needed team-certif
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants