[BO - Signalement] Enlever le "ne sait pas" pour les info non remplies

[numew]

Ticket

#2177 #2178 #2179

Description

• Possibilité de laisser la sélection vide dans les modale d'édition d'un signalement (pour conserver le fait qu'e l'information n'a pas été renseigné)
• Affichage des valeur "nsp" en "Ne sait pas" dans le BO sur les différents blocs de la fiche d'un signalement
• Retrait des affichages "N/C"/"Ne sait pas" dans le BO sur les différents blocs de la fiche d'un signalement quand l'information n'a pas été saisie

Tests

• Vérifier que l'affichage BO de la fiche signalement est correct et que les modale d'édition fonctionne correctement

[emilschn]

Quelques retours de relecture.
Tests incomplets pour l'instant

[hmeneuvrier]

Dans un signalement fait par un locataire, je change l'état des lieux en "je ne sais pas" et c'est "oui" qui reste affiché. Mmais si je change aussi le bail, alors la modification de l'état des lieux est prise en compte

[hmeneuvrier]

On a pareil avec certains éléments de situation du foyer comme "logement social", mais il me semble l'avoir corrigé dans la branche sur la suroccupation, donc quand ce sera mergé sur develop ça ira.

Par contre, peut-être qu'il faut récupérer develop sur ta branche, et vérifier pour chaque bloc que tous les éléments sont bien mis à jour tout le temps ?

[numew]

@hmeneuvrier, @emilschn vos retours sont traités

[emilschn]

Deux problèmes constatés :

• Dans Allocataire, si je remplis "oui" ou "non", j'ai toujours une valeur vide qui est affichée.
• "Bailleur averti" est à "OUI" ou "NON" dans l'affichage. Mais rien n'est sélectionné dans la modale.

[hmeneuvrier]

J'ai tout relu en comparant avec le tableau que j'ai fait, j'ai quelques petits commentaires

[hmeneuvrier]

Je ne comprends pas ce default ?

[numew]

ca retourne la valeur brute (si elle n'est ni 'piece_unique' ni 'plusieurs_pieces') tu veux dire que ca ne devrait pas arriver ? j'ai préféré prévoir le cas

[hmeneuvrier]

D'après https://docs.google.com/spreadsheets/d/1hvhNHY5ZS-MtgbEBo8WCUrj4aKExJqMxGANP3Z1zDEg/edit#gid=0, ça ne devrait pas arriver, mais je n'ai rien contre les précautions

[numew]

Je serais d'accord quand on aura les restrictions coté serveur 😉

[numew]

Deux problèmes constatés :

* Dans `Allocataire`, si je remplis "oui" ou "non", j'ai toujours une valeur vide qui est affichée.

* "Bailleur averti" est à "OUI" ou "NON" dans l'affichage. Mais rien n'est sélectionné dans la modale.

C'est corrigé

[emilschn]

OK relecture et tests :)
Belle persévérance :D

[hmeneuvrier]

@numew

Il y a encore un souci sur contact assurance, je choisis "pas d'assurance logement" et c'est nsp qui s'affiche, du coup quand je réédite, je suis sur rien :

[hmeneuvrier]

@numew
ça me chiffone qu'on puisse éditer Logement et mettre du vide alors que le choix entre pièce unique et plusieurs pièces est obligatoire côté front, c'est une demande de Mathilde ?

[numew]

@numew ça me chiffone qu'on puisse éditer Logement et mettre du vide alors que le choix entre pièce unique et plusieurs pièces est obligatoire côté front, c'est une demande de Mathilde ?

La demande de Mathilde c'est de ne pas afficher une information que l'utilisateur n'a pas saisie (Ex "Ne sait pas") pour un champ non renseigné. J'ai peut être extrapolé mais pour moi ca signifie que pour les signalement des anciens formulaire il faut pouvoir laisser ces valeur vide (imaginons si on veux modifier uniquement une infos (ex la superficie) si je ne laisse pas la possibilité de laisser vide ca va prendre une valeur par défaut de manière implicite.

[numew]

@numew

Il y a encore un souci sur contact assurance, je choisis "pas d'assurance logement" et c'est nsp qui s'affiche, du coup quand je réédite, je suis sur rien :

Merci, c'est corrigé

[sonarcloud]

Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

8 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[hmeneuvrier]

C'est tout bon pour moi !