-
Notifications
You must be signed in to change notification settings - Fork 57
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] Afficher le bouton de gestion du signalement en dehors du menu sur Pix Certif (PIX-16884). #11590
base: dev
Are you sure you want to change the base?
Conversation
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
Commentaire: Je me demande s'il n'y a pas (ici ou dans une autre PR) une amélioration à apporter pour faire en sorte que la disparition du bouton de gestion de signalement soit instantanée un fois le signalement traité. 🤔 |
certif/app/styles/components/session-supervising/candidate-in-list.scss
Outdated
Show resolved
Hide resolved
|
||
</div> | ||
{{#if @candidate.hasOngoingChallengeLiveAlert}} | ||
<PixButton @triggerAction={{this.askUserToHandleLiveAlert}} @variant="error"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion: Ajouter un loader lorsque l'action est en cours ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'étais partie pour le faire mais je me rend compte que je ne peut pas le faire de manière opti.
Déjà quand on déclenche ce bouton, ça fait apparaître une modale => donc en soit qu'il soit en loading n'est pas très utile, puisqu'on ne peut pas l'atteindre tant qu'on est dans cette modale.
Ensuite on effectue une action (valider/refuser le signalement, voire même fermer la modale)
On est censé à la suite de cette action arrêter le loader puisque l'action est finit.
Sauf qu'on se retrouve dans la même problématique qu'évoqué pdt la review en mob => le bouton reste visible tant que le polling n'a pas mis à jour des données.
Je me suis posé la question de laisser le loader actif tant que le polling à pas fait son job mais le composant n'a pas connaissance de l'action du polling et je ne peut pas mettre ma variable de loading à jour :/
Si vous avez une autre piste je prend mais du coup je laisse tel quel.
A savoir que le back enverra bouler si on tente de re-traiter un signalement déjà traité
<div class="session-supervising-candidate-in-list__full-name"> | ||
{{@candidate.lastName}} | ||
{{@candidate.firstName}} | ||
</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<div class="session-supervising-candidate-in-list__full-name"> | |
{{@candidate.lastName}} | |
{{@candidate.firstName}} | |
</div> | |
<div class="session-supervising-candidate-in-list__full-name"> | |
{{@candidate.lastName}} {{@candidate.firstName}} | |
</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
9a6e8ee
to
cf8782c
Compare
cf8782c
to
fee7cf2
Compare
🥞 Problème
Important
PR à review avant => #11583
Plusieurs remontées du support font état de candidats qui n’ont pas pu aller au bout de leur certification.
Lors de l’investigation, on se rend compte que les candidats avaient tous un signalement non traité par le surveillant sur une des questions de leur certification.
Le surveillant avait cliqué sur “Autoriser à reprendre” pensant que cela débloquerait son candidat.
Or actuellement un tag “Autorisé à reprendre” est prioritaire coté front et venait alors masquer le tag “Signalement en cours”.
Les surveillants pensaient donc avoir géré le signalement de leurs candidats.
🥓 Proposition
Après le tag “Autorisé à reprendre” retiré, on veut pouvoir rendre le bouton de gestion d'un signalement beaucoup plus visible que caché dans le menu.
🧃 Remarques
Lorsque j'ai ajouté le bouton en dehors du menu, j'ai testé d'afficher l'entièreté de ce que l'on pouvait trouver dans cette section.
En effet, il est possible d'y retrouver un bandeau de warning pour l'éligibilité à une complémentaire ou bien un tag pour le temps majoré.
Enregistrement.de.l.ecran.2025-03-06.a.15.29.01.mov
L'espace était beaucoup trop petit pour accueillir ce bouton supplémentaire.
D'après la maquette, la largeur des lignes est de 800px. J'en ai donc profité pour agrandir tout ça.
Pour des besoins de responsive, j'ai également fait un nettoyage de l'existant, tant coté html que css.
😋 Pour tester
Enregistrement.de.l.ecran.2025-03-06.a.15.42.16.mov
Jouer avec le responsive pour voir le comportement