Skip to content
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] Gestion de la recherche et d'affichage des bailleurs radiés #2398

Merged
merged 4 commits into from
Mar 28, 2024

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Mar 27, 2024

Ticket

#2390
#2391

Description

  • Apporte des améliorations à la fonctionnalité de recherche de bailleurs, avec un focus spécifique sur la gestion des bailleurs radiés. Il ne faut pas affiché la mention mais permettre la recherche sur le nom du bailleur radié et non sur la mention [Radié(e)]

  • Ajoute un texte d'aide au champ du bailleur avec auto-complétion

Changements apportés

  • Mise à jour URL auto-complétion afin de donner l'info si l'affichage doit être nettoyé ou pas
  • Mise à jour du contrôleur afin de filtrer et modifier les résultats de la recherche
  • Mise à jour afin de faire une recherche sur le bailleur radié depuis le FO
  • Ajout de test

Pré-requis

make create-db
make console app="import-bailleur"

Tests

  • Depuis le FO vérifier que le message d'aide s'affiche pour le locataire, service de secours, tiers particulier et pro.
  • Depuis le FO faite une recherche sur radié aucun résultat ne doit s'afficher
  • Depuis le FO faite une recherche sur ra tous les bailleurs qui ont la chaine ra dans leur nom doivent s'afficher , la mention ne doit pas s'afficher [Radié(e)]
  • Depuis le FO dans le département du 13 faite une recherche sur S.A choisissiez le bailleur qui est radié S.A. REGIONALE DE L'HABITAT puis enregistrer votre signalement et vérifier que le bailleur est bien enregistrer dans la fiche avec la mention [Radié(e)]
  • Depuis le BO vérifier que la recherche sur le bailleur fonctionne avec la mot radié et affiche la mention [Radié(e)]

@sfinx13 sfinx13 changed the title [FO] Gestion de la recherche et d'affiche des bailleurs radiés [FO] Gestion de la recherche et d'affichage des bailleurs radiés Mar 27, 2024
@sfinx13 sfinx13 requested review from emilschn, hmeneuvrier and numew and removed request for emilschn March 27, 2024 14:11
Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Relecture et tests OK

Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Une question de code mais j'approuve

src/Repository/BailleurRepository.php Outdated Show resolved Hide resolved
@sfinx13 sfinx13 force-pushed the bugfix/2391-2390-feedback-bailleurs branch from 86b0498 to 9587cc4 Compare March 28, 2024 13:02
Copy link

sonarcloud bot commented Mar 28, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarCloud

@numew numew merged commit eb573b7 into develop Mar 28, 2024
3 checks passed
@hmeneuvrier hmeneuvrier deleted the bugfix/2391-2390-feedback-bailleurs branch April 11, 2024 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants