-
Notifications
You must be signed in to change notification settings - Fork 0
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
[FO - Signalement] Pour les tiers, la réponse de l'assurance est un champ facultatif #2725
Conversation
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.
Concernant le cadre de cette PR, pour moi les tests et la relecture sont OK
Mais je me suis aperçue que dans le récapitulatif, on a le récap de la procédure seulement pour bailleur_occupant, locataire et bailleur, Je pense qu'il faudrait enlever la condition maintenant qu'on a ajouté ces infos pour les autres profils tiers, non ?
Je viens de faire le test en enlevant le v-if ligne 136 de SignalementFormOverview, ça fonctionne bien
C'est fait ! Merci ! |
Quality Gate passedIssues Measures |
OK pour moi |
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.
Ok, est-ce qu'on peux en profiter pour ajouter le contrôle des valeurs sur le champ assurance ?
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.
Ok
Ticket
#2275
Description
Dans le formulaire de signalement, la réponse de l'assurance est un champ facultatif pour les tiers.
Pré-requis
npm run watch
Tests