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 - Signalement] Ajouter un champ sur la composition du logement #3530

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

hmeneuvrier
Copy link
Collaborator

@hmeneuvrier hmeneuvrier commented Jan 8, 2025

Ticket

#3445

Description

Modif formulaire FO - Ecran composition-logement

  • Ajouter la question Dont combien d'enfants ?
  • Modifier la question composition_logement_enfants pour ne la faire apparaître que si on a plus de 0 à la question précédente (on garde la règle dans l'algo : si la réponse à cette question est oui, on multiplie )

Affichage dans le BO - Onglet foyer

  • Dans la partie "infos sur le logement",
    • On garde le nombre de personnes, et on met à la suite :
    • Ajouter Dont nb. d'enfants : {{valeur}}

Changements apportés

  • Modifications dans les 6 json du formulaire + ajout dans le récap du signalement
  • Ajout de ces infos dans quelques fixtures
  • Ajout de ces infos dans les DTO, Model et Factory, Controller
  • Ajout des infos dans l'export de la liste de signalement
  • Ajout des infos dans les twig d'affichage BO, d'édition BO, de page de suivi, et d'export pdf du signalement
  • Mise à jour des tests

Pré-requis

make load-fixtures
npm run watch

Tests

  • Déposer un signalement en service secours, et vérifier qu'il y a la question sur le nombre d'enfants, mais toujours pas sur les enfants de moins de 6 ans
  • Vérifier l'affichage et l'édition dans le BO

⚠️ Faire les tests suivants avec un autre profil

  • Déposer un signalement, et vérifier l'affichage de la question sur le nombre d'enfants qui conditionne l'affichage de la question sur les enfants de moins de 6 ans
  • Dans le récap du signalement, vérifier qu'il y a les infos
  • Après l'envoi du signalement, vérifier l'affichage de la fiche dans le BO
  • Vérifier l'édition des infos dans le BO
  • Vérifier leur affichage dans l'export pdf
  • Vérifier leur affichage dans la page de suivi
  • Vérifier leur affichage lors de l'appel à l'API
  • Vérifier l'apparition des colonnes dans l'export de la liste de signalements

@hmeneuvrier hmeneuvrier marked this pull request as ready for review January 8, 2025 13:52
@hmeneuvrier hmeneuvrier changed the title add nb enfants form #3445 [FO - Signalement] Ajouter un champ sur la composition du logement Jan 8, 2025
@hmeneuvrier hmeneuvrier force-pushed the feature/3445-fo---signalement-ajouter-champs-composition-logement branch from 5a10673 to 88cd00f Compare January 8, 2025 15:55
@hmeneuvrier hmeneuvrier force-pushed the feature/3445-fo---signalement-ajouter-champs-composition-logement branch from 88cd00f to 631e2f0 Compare January 10, 2025 07:55
@@ -189,4 +190,13 @@ private function stringToBool(?string $value): ?bool

return null;
}

private function stringToInt(?string $value): ?int
Copy link
Collaborator

Choose a reason for hiding this comment

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

fonction plus globale à mettre en place ? je suis surpris qu'on n'en ait besoin qu'ici :)

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.

Des commentaires / questions ; pas encore de tests

@hmeneuvrier hmeneuvrier force-pushed the feature/3445-fo---signalement-ajouter-champs-composition-logement branch from fc6a958 to 50c2450 Compare January 10, 2025 10:02
@hmeneuvrier hmeneuvrier force-pushed the feature/3445-fo---signalement-ajouter-champs-composition-logement branch from 8ae6d22 to d07a27e Compare January 10, 2025 14:29
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
13.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

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.

3 participants