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

[Front] Désordres dans la page signalement BO #2089

Merged
merged 16 commits into from
Jan 23, 2024

Conversation

hmeneuvrier
Copy link
Collaborator

@hmeneuvrier hmeneuvrier commented Jan 5, 2024

Ticket

#1811

Description

Affichage des désordres dans la page de signalement BO

Changements apportés

  • Modification des fixtures, ajout de labels aux précisions et dé-doublage de certaines précisions (par exemple pour ne pas avoir la même précision quand on répond "non" ou "je ne sais pas")
  • Correction de quelques services de traitement des désordres suite au dé-doublage de certaines précisions
  • Modification du SignalementController pour envoyer les bonnes données au twig
  • Création du template twig pour afficher les désordres
  • Modification du FileFactory pour supprimer les suffixes inuntiles des photos
  • Ajout d'une fonction getPhotosBySlug dans le SignalementManager

⚠️ La mise à jour des qualifications, tags et scores depuis la fiche du BO seront traités dans le ticket #2073

Pré-requis

make composer
make load-fixtures

Tests

  • Faire un ou des signalements avec le nouveau formulaire en choisissant des profils variés, des désordres variés, et des réponses toutes aussi variées. Ajout des photos ou fichiers quand c'est possible (notamment le disgnostique plomb)
  • Vérifier l'affichage des désordres dans la page de signalement

@hmeneuvrier hmeneuvrier changed the base branch from feature/1546-back-score-criticite to feature/1547-back-qualification January 11, 2024 10:32
@hmeneuvrier hmeneuvrier force-pushed the feature/1811-show-desordres-in-signalement-bo branch from 129e638 to a21caf2 Compare January 11, 2024 10:38
@hmeneuvrier hmeneuvrier changed the title Feature/1811 show desordres in signalement bo [Front] Désordres dans la page signalement BO Jan 11, 2024
@hmeneuvrier hmeneuvrier marked this pull request as ready for review January 11, 2024 16:33
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.

Correction a faire dans l'affichage des photo (qui sont les une au dessus des autre plutôt que cote à cote) + l'enregistrement du champ autre pour les nuisible coté batiment.

Pour le reste tout me semble bon, tests ok

src/Controller/Back/SignalementController.php Outdated Show resolved Hide resolved
src/Manager/SignalementManager.php Show resolved Hide resolved
@hmeneuvrier hmeneuvrier force-pushed the feature/1547-back-qualification branch from 2ecc22d to aa03cda Compare January 15, 2024 08:17
@hmeneuvrier hmeneuvrier force-pushed the feature/1811-show-desordres-in-signalement-bo branch from df72cc2 to 2fd2028 Compare January 15, 2024 10:21
@hmeneuvrier
Copy link
Collaborator Author

@emilschn Concernant les tirets, as-tu mis à jour les fixtures ? (c'est le même truc partout)

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

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

quelques retours d'affichage

@emilschn
Copy link
Collaborator

emilschn commented Jan 15, 2024

J'avais 1 pièce pour 3 personnes et 323 m². J'ai le désordre et la qualification "Suroccupation".
Je suis passé à 3 pièces, mais ça a gardé le désordre et la qualif.

[edit : vu ensemble, sera fait dans un prochain ticket]

@emilschn
Copy link
Collaborator

emilschn commented Jan 15, 2024

image
Je ne pense pas que ce soit lié à ce que tu as fait, mais je l'ai eu, donc je signale... :)
Peut-être lié à la PR précédente, c'est lié à la NDE.
J'ai fait le signalement en tant que voisinage. Donc je n'ai pas mis de date d'entrée

@emilschn
Copy link
Collaborator

@emilschn Concernant les tirets, as-tu mis à jour les fixtures ? (c'est le même truc partout)

En effet, c'est ok ! J'ai retiré mes coms.

@hmeneuvrier
Copy link
Collaborator Author

Je ne pense pas que ce soit lié à ce que tu as fait, mais je l'ai eu, donc je signale... :) Peut-être lié à la PR précédente, c'est lié à la NDE. J'ai fait le signalement en tant que voisinage. Donc je n'ai pas mis de date d'entrée

Oui bien vu ! je corrige dans cette PR du coup

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

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

OK pour moi
Je laisse @sfinx13 revoir en fonction de ses retours précédents.

@hmeneuvrier hmeneuvrier force-pushed the feature/1547-back-qualification branch from 2a87889 to c3dea3f Compare January 18, 2024 14:33
@sfinx13 sfinx13 marked this pull request as draft January 18, 2024 15:01
@hmeneuvrier hmeneuvrier force-pushed the feature/1547-back-qualification branch from fead63c to 54b5842 Compare January 22, 2024 13:48
@hmeneuvrier hmeneuvrier force-pushed the feature/1811-show-desordres-in-signalement-bo branch 2 times, most recently from 2d63d42 to 62a06f7 Compare January 22, 2024 17:07
@hmeneuvrier hmeneuvrier changed the base branch from feature/1547-back-qualification to develop January 22, 2024 17:07
@hmeneuvrier hmeneuvrier marked this pull request as ready for review January 22, 2024 17:07
@hmeneuvrier hmeneuvrier force-pushed the feature/1811-show-desordres-in-signalement-bo branch from 62a06f7 to cf9f0d0 Compare January 23, 2024 10:06
Copy link

sonarcloud bot commented Jan 23, 2024

Quality Gate Failed Quality Gate failed

Failed conditions

3.7% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@numew numew merged commit e30a417 into develop Jan 23, 2024
2 of 3 checks passed
@hmeneuvrier hmeneuvrier deleted the feature/1811-show-desordres-in-signalement-bo branch January 30, 2024 14: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

4 participants