[FO - Signalement] Autocomplétion Bailleur social #2329

sfinx13 · 2024-03-11T15:53:03Z

Ticket

Description

Ajout d'une liste de bailleurs en auto-complétion si c'est un logement social.

Changements apportés

Ajout d'une nouvelle table "bailleur".
Ajout d'une nouvelle commande sheel pour pousser le csv bailleur
Ajout d'une commande pour importer la liste des bailleurs.
Ajout d'une nouvelle route
Ajout d'un nouveau composant d'autocomplétion pour le formulaire.
Ajout du système d'auto-complétion côté BO si c'est un logement social.
Correction de noms du territoire
Passer le service ZipcodeProvider en statique

Prérequis

make create-db
make npm-build

Le fichier est déjà sur le bucket

Tests

assets/vue/components/signalement-form/components/SignalementFormAutocomplete.vue

emilschn · 2024-03-12T15:10:43Z

assets/vue/components/signalement-form/components/SignalementFormAutocomplete.vue

+    />
+
+    <div class="fr-grid-row fr-background-alt--blue-france fr-text-label--blue-france fr-autocomplete-group">
+      <div class="fr-col-12 fr-p-3v fr-text-label--blue-france fr-autocomplete-suggestion"


possible de remplacer par un <select> en multiline, pour pouvoir utiliser le clavier nativement ?
j'avais fait comme ça pour les adresses sur Punaises, même si je crois qu'il y a des retours dessus qu'on n'a pas encore traité :)

Vu en stand up

implémentation de la navigation au clavier OK

Fermeture de la liste sur un clic extérieur à la zone de suggestion

Implémentation pour le rendre 100% accessible avec les balise aria demande plus de temps (plusieurs attributs à gérer)
Voir => https://alphagov.github.io/accessible-autocomplete/

Prise en charge dans le ticket des attributs (aria-*), tabindex etc...
#2341

assets/vue/components/signalement-form/components/SignalementFormAutocomplete.vue

src/Command/ImportBailleurCommand.php

emilschn

Quelques retours. Pas encore testé.

numew

Quelque remarques en attendant le changement de structure

assets/vue/components/signalement-form/components/SignalementFormAutocomplete.vue

src/Service/Import/Bailleur/BailleurLoader.php

src/Service/Signalement/SignalementBuilder.php

numew

Quelques retours et propositions.

1 Test KO : Si on modifie la valeur dan le BO en rentrant une valeur libre l'id bailleur n'est pas modifié

Petites remarque :

FO/BO : Quand on passe le champ autocomplete a vide les proposition reste affichés
Une des valeurs a un caractère anormal mais je pense que le soucis est sur le CSV

src/Entity/Bailleur.php

src/Repository/BailleurRepository.php

src/Entity/Signalement.php

src/Manager/SignalementManager.php

sfinx13 · 2024-03-21T17:35:16Z

@numew
Ok pour ces retours

Quelques retours et propositions.
1 Test KO : Si on modifie la valeur dan le BO en rentrant une valeur libre l'id bailleur n'est pas modifié
Petites remarque :
FO/BO : Quand on passe le champ autocomplete a vide les proposition reste affichés
Une des valeurs a un caractère anormal mais je pense que le soucis est sur le CSV

emilschn

Un retour de test bloquant
En tant que particulier sans logement social, on ne me propose pas la liste des bailleurs OK
Mais par contre, je ne peux pas passer à l'écran suivant KO.

numew · 2024-03-22T17:01:58Z

@numew Ok pour ces retours

Quelques retours et propositions.
1 Test KO : Si on modifie la valeur dan le BO en rentrant une valeur libre l'id bailleur n'est pas modifié
Petites remarque :
FO/BO : Quand on passe le champ autocomplete a vide les proposition reste affichés
Une des valeurs a un caractère anormal mais je pense que le soucis est sur le CSV

Ok pour le test KO qui a été corrigé, je n'ai pas vu de différence sur les autres points

src/Entity/BailleurTerritory.php

sfinx13 · 2024-03-25T14:11:11Z

@numew Ok pour ces retours

Quelques retours et propositions.
1 Test KO : Si on modifie la valeur dan le BO en rentrant une valeur libre l'id bailleur n'est pas modifié
Petites remarque :
FO/BO : Quand on passe le champ autocomplete a vide les proposition reste affichés
Une des valeurs a un caractère anormal mais je pense que le soucis est sur le CSV

Ok pour le test KO qui a été corrigé, je n'ai pas vu de différence sur les autres points

@numew la cette fois ci c'est bon :-D

sfinx13 · 2024-03-25T14:11:56Z

Un retour de test bloquant En tant que particulier sans logement social, on ne me propose pas la liste des bailleurs OK Mais par contre, je ne peux pas passer à l'écran suivant KO.

Retour bloquant traité @hmeneuvrier @numew

hmeneuvrier · 2024-03-25T15:41:06Z

Test KO
[FO] En tant que déclarant sans avoir coché "logement social", la liste des bailleurs ne m'est pas proposée en autocomplétion.

Tous les autres sont OK

sonarcloud · 2024-03-25T16:00:49Z

Quality Gate passed

Issues
10 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
2.2% Duplication on New Code

See analysis details on SonarCloud

tools/wiremock/src/Resources/Signalement/questions_profile_locataire.json

hmeneuvrier · 2024-03-25T16:07:30Z

Tests OK maintenant

hmeneuvrier

OK pour moi

numew

Ok

sfinx13 force-pushed the feature/2215-bailleur-autocompletion branch from f0b2cef to 56742be Compare March 11, 2024 16:12

sfinx13 requested review from emilschn, numew and hmeneuvrier March 11, 2024 16:15

emilschn reviewed Mar 12, 2024

View reviewed changes

assets/vue/components/signalement-form/components/SignalementFormAutocomplete.vue Outdated Show resolved Hide resolved

emilschn reviewed Mar 12, 2024

View reviewed changes

assets/vue/components/signalement-form/components/SignalementFormAutocomplete.vue Outdated Show resolved Hide resolved

emilschn reviewed Mar 12, 2024

View reviewed changes

src/Command/ImportBailleurCommand.php Outdated Show resolved Hide resolved

emilschn requested changes Mar 12, 2024

View reviewed changes

numew reviewed Mar 12, 2024

View reviewed changes

sfinx13 force-pushed the feature/2215-bailleur-autocompletion branch from b957ef4 to d981700 Compare March 18, 2024 10:46

numew reviewed Mar 20, 2024

View reviewed changes

sfinx13 added 8 commits March 21, 2024 18:57

implement autocomplete list bailleur #2215

77b19e9

navigate autocomplete from keyboard and apply some feedback #2215

4fde264

update id component #2215

37bc66b

remove useless variables #2215

e54543c

fix test #2215

555053d

update structure and apply some ux fixes #2215

2533e35

update test #2215

9292641

fix conflicts #2215

f8f9771

sfinx13 force-pushed the feature/2215-bailleur-autocompletion branch from 0e8a26b to f8f9771 Compare March 21, 2024 19:49

update test #2215

f08ba8e

emilschn requested changes Mar 22, 2024

View reviewed changes

numew reviewed Mar 22, 2024

View reviewed changes

src/Entity/BailleurTerritory.php Show resolved Hide resolved

apply some fixes #2215

4f3f3e1

update test #2215

1f8097b

sfinx13 added 2 commits March 25, 2024 15:25

apply some sonar feedback #2215

ea29ce2

remove comma #2215

0f7627a

update services secours json #2215

f2aec70

hmeneuvrier reviewed Mar 25, 2024

View reviewed changes

tools/wiremock/src/Resources/Signalement/questions_profile_locataire.json Show resolved Hide resolved

tools/wiremock/src/Resources/Signalement/questions_profile_locataire.json Show resolved Hide resolved

hmeneuvrier approved these changes Mar 25, 2024

View reviewed changes

numew approved these changes Mar 25, 2024

View reviewed changes

numew merged commit 8b1e3bb into develop Mar 25, 2024
3 checks passed

hmeneuvrier deleted the feature/2215-bailleur-autocompletion branch April 11, 2024 13:17

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[FO - Signalement] Autocomplétion Bailleur social #2329

[FO - Signalement] Autocomplétion Bailleur social #2329

sfinx13 commented Mar 11, 2024 •

edited

Loading

emilschn Mar 12, 2024

sfinx13 Mar 18, 2024 •

edited

Loading

sfinx13 Mar 18, 2024 •

edited

Loading

emilschn left a comment

numew left a comment

numew left a comment

sfinx13 commented Mar 21, 2024

emilschn left a comment

numew commented Mar 22, 2024

sfinx13 commented Mar 25, 2024

sfinx13 commented Mar 25, 2024 •

edited

Loading

hmeneuvrier commented Mar 25, 2024

sonarcloud bot commented Mar 25, 2024

hmeneuvrier commented Mar 25, 2024

hmeneuvrier left a comment

numew left a comment

[FO - Signalement] Autocomplétion Bailleur social #2329

[FO - Signalement] Autocomplétion Bailleur social #2329

Conversation

sfinx13 commented Mar 11, 2024 • edited Loading

Ticket

Description

Changements apportés

Prérequis

Tests

emilschn Mar 12, 2024

Choose a reason for hiding this comment

sfinx13 Mar 18, 2024 • edited Loading

Choose a reason for hiding this comment

sfinx13 Mar 18, 2024 • edited Loading

Choose a reason for hiding this comment

emilschn left a comment

Choose a reason for hiding this comment

numew left a comment

Choose a reason for hiding this comment

numew left a comment

Choose a reason for hiding this comment

sfinx13 commented Mar 21, 2024

emilschn left a comment

Choose a reason for hiding this comment

numew commented Mar 22, 2024

sfinx13 commented Mar 25, 2024

sfinx13 commented Mar 25, 2024 • edited Loading

hmeneuvrier commented Mar 25, 2024

sonarcloud bot commented Mar 25, 2024

Quality Gate passed

hmeneuvrier commented Mar 25, 2024

hmeneuvrier left a comment

Choose a reason for hiding this comment

numew left a comment

Choose a reason for hiding this comment

sfinx13 commented Mar 11, 2024 •

edited

Loading

sfinx13 Mar 18, 2024 •

edited

Loading

sfinx13 Mar 18, 2024 •

edited

Loading

sfinx13 commented Mar 25, 2024 •

edited

Loading