[Front - Formulaire] Page BO signalement pour nouveau formulaire #1788

emilschn · 2023-10-02T13:43:01Z

Ticket

#1548

Description

Page BO signalement pour ceux créés via nouveau formulaire

Tests

Tests feature flipping

Désactiver FEATURE_NEW_FORM_ENABLE
Tester l'affichage des anciens signalement
Tester l'édition des anciens signalement

hmeneuvrier · 2023-10-18T10:19:18Z

Edition d'un signalement existant :
J'ouvre la modale d'ajout d'étiquette, j'en ajoute une et je n'arrive plus à fermer la modale.

Edition des corrdonnées du bailleur, l'adresse n'est pas découpée.

J'ouvre la modale de situation du foyer, et j'enregistre sans avoir rien modifié, j'ai l'erreur suivante (également en modifiant la date de naissance) (et pareil pour la partie Informations sur le logement et Procédures et démarches) :

hmeneuvrier · 2023-10-18T10:31:11Z

Nouveau signalement en locataire

j'ai mis un DPE dans le parcours, mais je ne le vois pas en cliquant sur le bouton

hmeneuvrier · 2023-10-18T12:34:16Z

Signalement en tant que bailleur occupant

Le nom du proprio n'est pas répercuté sur l'encart "bailleur" contrairement au mail et au prénom, et il n'y a pas N/R pour le tel secondaire de l'occupant

Il manque des contraintes dans les modales, par exemple on peut remplacer le nombre de personnes (int) par du texte :

J'ai ajouté le montant de l'allocation dans la situation du foyer, mais ça n'apparait pas dans l'encart après validation.

public/js/const.min.js

public/js/app.js

src/Controller/Back/SignalementEditController.php

src/Dto/Request/Signalement/CoordonneesTiersRequest.php

sfinx13 · 2023-10-18T13:02:51Z

src/Dto/Request/Signalement/InformationsLogementRequest.php

+
+class InformationsLogementRequest
+{
+    public function __construct(


Vérifier si'il y'a des contraintes à ajouter

sfinx13 · 2023-10-18T13:03:09Z

src/Dto/Request/Signalement/ProcedureDemarchesRequest.php

+class ProcedureDemarchesRequest
+{
+    public function __construct(
+        private readonly ?string $isProprioAverti = null,


Vérifier s'il y'a des contraintes à ajouter

sfinx13 · 2023-10-18T13:03:34Z

src/Dto/Request/Signalement/SituationFoyerRequest.php

+
+class SituationFoyerRequest
+{
+    public function __construct(


Vérifier s'il y'a des contraintes à ajouter

src/Manager/SignalementManager.php

hmeneuvrier · 2023-10-18T14:12:48Z

Signalement en tant que voisin

composition du logement, est-ce qu'on laisse nsp au lieu de Ne sait pas ?
pour la superficie il faudrait N/C

coordonnées du foyer mettre N/R ou N/C plutôt que rien (et en choisir un seul, car là ou a soit l'un soit l'autre)

possible de ne pas faire apparaitre du tout le champ structure dans la modale ?

pour la situation du foyer, il est possible de mettre un montant d'allocation ou un numéro d'allocataire alors qu'on a dit "non" pour allocataire, et du coup ça n'apparait pas dans l'encart (cf plus haut). C'est un peu perturbant. On pourrait ne pas afficher ces champs dans la modale tant qu'on n'a pas mis CAF ou MSA dans la selectbox plus haut

templates/back/signalement/view/address-qualifications.html.twig

templates/back/signalement/view/edit-modals/edit-procedure-demarches.html.twig

public/js/const.min.js

src/Manager/SignalementManager.php

templates/back/signalement/view.html.twig

templates/back/signalement/view/information.html.twig

emilschn · 2023-10-19T15:21:42Z

Edition d'un signalement existant : J'ouvre la modale d'ajout d'étiquette, j'en ajoute une et je n'arrive plus à fermer la modale.

Edition des corrdonnées du bailleur, l'adresse n'est pas découpée.

J'ouvre la modale de situation du foyer, et j'enregistre sans avoir rien modifié, j'ai l'erreur suivante (également en modifiant la date de naissance) (et pareil pour la partie Informations sur le logement et Procédures et démarches) :

Ouverture de modale pour étiquette : il faut rappuyer sur le bouton comme actuellement (c'est pas idéal, mais je n'ai pas modifié le comportement)

Edition des coordonnées du bailleur : non reproduit

Edition modale situation foyer : corrigé !

emilschn · 2023-10-20T10:39:54Z

Nouveau signalement en locataire

j'ai mis un DPE dans le parcours, mais je ne le vois pas en cliquant sur le bouton

Cette tâche n'a pas encore été faite : #1668

emilschn · 2023-10-20T13:12:21Z

Signalement en tant que voisin

composition du logement, est-ce qu'on laisse nsp au lieu de Ne sait pas ? pour la superficie il faudrait N/C

coordonnées du foyer mettre N/R ou N/C plutôt que rien (et en choisir un seul, car là ou a soit l'un soit l'autre)

possible de ne pas faire apparaitre du tout le champ structure dans la modale ?

pour la situation du foyer, il est possible de mettre un montant d'allocation ou un numéro d'allocataire alors qu'on a dit "non" pour allocataire, et du coup ça n'apparait pas dans l'encart (cf plus haut). C'est un peu perturbant. On pourrait ne pas afficher ces champs dans la modale tant qu'on n'a pas mis CAF ou MSA dans la selectbox plus haut

@mathildepoulpux
Est-ce que tu as un avis sur le "nsp" en haut (par rapport aux nsp plus bas) ?
Et est-ce qu'on rend les formulaires du BO dynamiques (avec des champs qui s'affichent / se masquent) ?

mathildepoulpux · 2023-10-24T10:09:18Z

Signalement en tant que voisin
composition du logement, est-ce qu'on laisse nsp au lieu de Ne sait pas ? pour la superficie il faudrait N/C

coordonnées du foyer mettre N/R ou N/C plutôt que rien (et en choisir un seul, car là ou a soit l'un soit l'autre)
possible de ne pas faire apparaitre du tout le champ structure dans la modale ?
pour la situation du foyer, il est possible de mettre un montant d'allocation ou un numéro d'allocataire alors qu'on a dit "non" pour allocataire, et du coup ça n'apparait pas dans l'encart (cf plus haut). C'est un peu perturbant. On pourrait ne pas afficher ces champs dans la modale tant qu'on n'a pas mis CAF ou MSA dans la selectbox plus haut

@mathildepoulpux Est-ce que tu as un avis sur le "nsp" en haut (par rapport aux nsp plus bas) ? Et est-ce qu'on rend les formulaires du BO dynamiques (avec des champs qui s'affichent / se masquent) ?

Je mettrais "ne sait pas" en toute lettres et en enlevant le label. Ca a pas trop de sens d'attirer l'oeil sur des infos absentes.
Par contre non pour le formulaire dynamique, j'ai peur que ça perturbe les users d'avoir des champs variables d'un signalement à l'autre (+ s'ils veulent ajouter une info, ils seront pas forcément où la mettre si le champ est masqué).

emilschn · 2023-10-30T17:27:11Z

@sfinx13 @hmeneuvrier j'ai traité vos retours ! :)

sfinx13

Pris en charge feature fiipping sur:

Boutons modifier (pour éviter l'édition via les modales)
Bouton éditer fiche signalement et liste
Accès à la page d'édition via url

https://mattermost.incubateur.net/betagouv/pl/xfi7sbpj9bdr7kc3mooet54wda

src/Validator/TelephoneFormatValidator.php

sonarcloud · 2023-11-24T11:04:33Z

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
21 Code Smells

No Coverage information
0.0% Duplication

emilschn force-pushed the feature/1548-new-form-bo-page branch 4 times, most recently from bcf4fbe to 040c3d4 Compare October 9, 2023 11:55

emilschn changed the base branch from develop to feature/1544-save-signalement October 9, 2023 15:19

emilschn force-pushed the feature/1548-new-form-bo-page branch 2 times, most recently from 57045bb to c6924a6 Compare October 12, 2023 12:53

emilschn changed the base branch from feature/1544-save-signalement to develop October 12, 2023 12:53

emilschn force-pushed the feature/1548-new-form-bo-page branch from 96f6dce to 39ecb98 Compare October 13, 2023 07:50

emilschn changed the title ~~[WIP] [Front - Formulaire] Page BO signalement pour nouveau formulaire~~ [Front - Formulaire] Page BO signalement pour nouveau formulaire Oct 17, 2023

emilschn requested a review from hmeneuvrier October 17, 2023 15:14

emilschn marked this pull request as ready for review October 17, 2023 15:16

sfinx13 self-requested a review October 18, 2023 12:56

sfinx13 requested changes Oct 18, 2023

View reviewed changes

src/Manager/SignalementManager.php Outdated Show resolved Hide resolved

hmeneuvrier reviewed Oct 18, 2023

View reviewed changes

sfinx13 mentioned this pull request Oct 19, 2023

[Back] Créer un service pour la transformation des valeurs d'entrée du formulaire #1837

Merged

2 tasks

emilschn force-pushed the feature/1548-new-form-bo-page branch from 7036802 to 5db1804 Compare October 19, 2023 14:18

emilschn force-pushed the feature/1548-new-form-bo-page branch from c0c2ee9 to baad6d8 Compare October 30, 2023 13:41

sfinx13 requested changes Nov 2, 2023

View reviewed changes

sfinx13 reviewed Nov 2, 2023

View reviewed changes

src/Validator/TelephoneFormatValidator.php Show resolved Hide resolved

emilschn and others added 25 commits November 24, 2023 11:27

change way to unserialize form request

dc463e8

mix old and new form information in bo

6a7533e

use serializer for bo form request

78e553a

edit coordonnees tiers in bo

b1e0d46

save lien with occupant for all the tiers

e92e307

bo edit coordonnees of foyer

d834e85

bo edit coordoonnees of bailleur

35dce51

bo signalement edit informations logement

0eb4b22

wip save informations logement

b46cee8

wip bo save signalement situation foyer

2ef4a57

wip bo signalement edit procedures et demarches

437af03

missing file with last commit

de35a9d

fixes after first tests

f92d25d

add control on superficie

026fa02

add signalement input mapper #1835

d0c7db5

remove edition of old js file, add new js component

c1c3986

add validation constraints

2719f83

add telephone validator for french format

8e1181a

display Ne sait pas, avoid saving empty birthdate

0859b55

add constraint on phone numbers

7e9a8e1

fix display bo signalement

94f2d17

fix phpstan type error

bdc7923

test on nb occupants

c30f5f4

add feature flipping for edit signalement

cbe2d15

typo phpdoc

77dca38

emilschn force-pushed the feature/1548-new-form-bo-page branch from 6eb965a to 77dca38 Compare November 24, 2023 10:28

fix phpdoc #1548

7ec2c1f

emilschn marked this pull request as ready for review November 27, 2023 13:10

numew merged commit c2c28d2 into develop Nov 27, 2023
5 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Front - Formulaire] Page BO signalement pour nouveau formulaire #1788

[Front - Formulaire] Page BO signalement pour nouveau formulaire #1788

emilschn commented Oct 2, 2023 •

edited

Loading

hmeneuvrier commented Oct 18, 2023

hmeneuvrier commented Oct 18, 2023 •

edited

Loading

hmeneuvrier commented Oct 18, 2023 •

edited

Loading

sfinx13 Oct 18, 2023

sfinx13 Oct 18, 2023

sfinx13 Oct 18, 2023

hmeneuvrier commented Oct 18, 2023

emilschn commented Oct 19, 2023

emilschn commented Oct 20, 2023

emilschn commented Oct 20, 2023

mathildepoulpux commented Oct 24, 2023

emilschn commented Oct 30, 2023

sfinx13 left a comment

sonarcloud bot commented Nov 24, 2023

[Front - Formulaire] Page BO signalement pour nouveau formulaire #1788

[Front - Formulaire] Page BO signalement pour nouveau formulaire #1788

Conversation

emilschn commented Oct 2, 2023 • edited Loading

Ticket

Description

Tests

Tests feature flipping

hmeneuvrier commented Oct 18, 2023

hmeneuvrier commented Oct 18, 2023 • edited Loading

hmeneuvrier commented Oct 18, 2023 • edited Loading

sfinx13 Oct 18, 2023

Choose a reason for hiding this comment

sfinx13 Oct 18, 2023

Choose a reason for hiding this comment

sfinx13 Oct 18, 2023

Choose a reason for hiding this comment

hmeneuvrier commented Oct 18, 2023

emilschn commented Oct 19, 2023

emilschn commented Oct 20, 2023

emilschn commented Oct 20, 2023

mathildepoulpux commented Oct 24, 2023

emilschn commented Oct 30, 2023

sfinx13 left a comment

Choose a reason for hiding this comment

sonarcloud bot commented Nov 24, 2023

emilschn commented Oct 2, 2023 •

edited

Loading

hmeneuvrier commented Oct 18, 2023 •

edited

Loading

hmeneuvrier commented Oct 18, 2023 •

edited

Loading