Permet à la viseuse de changer la décision de l'instruction #1479
Conversation
alemangui
changed the title
There was a problem hiding this comment.
Stp faire un search and replace dans le code pour overriden -> overridden 😅
J'imagine c'est hors du scope de cette PR, mais l'alignement du message final n'est pas bon je crois - le l'attende à droit pour indiquer que c'est écrit par l'équipe et non pas le pro
| ) | ||
| const instructorName = computed(() => { | ||
| if (!declaration.value?.instructor) return "-" | ||
| return `${declaration.value.instructor.firstName || ""} ${declaration.value.instructor.lastName || ""}` |
There was a problem hiding this comment.
suggestion de refacto dans une autre PR: avoir une méthode qui fait le format du nom (c'est refait dans plusieurs endroits selon un recherche pour firstName) et aussi une méthode pour textOrDash ou qqch
There was a problem hiding this comment.
Bien vu. OK pour le faire dans une autre PR
|
@hfroot commit 7d8b795 adresse le souci avec le Le souci venait de la page d'instruction qui mettait à tort 15 jours de réponse pour des statuts comme "approbation" qui n'en avaient pas besoin. Il n'y a pas de conséquences métier, mais c'est moche. J'ai mis à jour frontend/src/views/InstructionPage/DecisionTab.vue pour mettre les jours à null dans le cas d'une approbation ou un refus. À noter que les déclarations déjà en visa auront toujours ce problème.
|
Closes #1267
Contexte
La viseuse peut aujourd'hui valider ou refuser une demande de visa, mais pas la modifier. Cette PR vise à ajouter cette fonctionnalité.
Scope
La page de validation aura un nouveau bouton : « Modifier la décision » qui permettra de surcharger celle de l'instructrice :
Screencast.from.2025-01-21.11-18-47.mp4
Une fois la décision surchargée, on voit dans la page la décision précédente (celle de l'instructrice) en format
strikethrough. Il y a ici la possibilité de revenir en arrière avec le bouton « Annuler la modification »Screencast.from.2025-01-21.11-21-09.mp4
Pour que les modifications soient effectuées, il faut valider la demande. Le message change pour signaler que la validation du visa applique aussi les modifications :
Refuser le visa signifie perdre les modifications faites :
Technique
La modification se fait au niveau de la view
DeclarationAcceptVisaView. Elle effectuait déjà une modification du commentaire, donc ça m'a semblé approprié de simplement éteindre cette fonctionnalité.Côté frontend j'ai opté pour un modal pour s'assurer que les différentes validations continuent à s'appliquer : par exemple le fait de devoir choisir une raison lors qu'on fait une observation.