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

[Algo] Modifier la qualification assurantielle #2154

Merged
merged 3 commits into from
Jan 30, 2024

Conversation

hmeneuvrier
Copy link
Collaborator

@hmeneuvrier hmeneuvrier commented Jan 29, 2024

Ticket

#2150

Description

Supprimer la tranche score = 0
Revoir la tranche score entre 0 et 10 pour inclure l'assurantiel (voir ci-dessous)
Retirer la qualification "Assurantiel" dans le cas où Dégâts des eaux = NON -> à jour dans le tableau lignes 115 à 117
Penser à l'édition BO de "Procédure et démarche" -> si on bascule Contact assurance en Oui ou en Non

Changements apportés

  • Mise à jour des fixtures
  • Mise à jour du SignalementManager pour recalculer les qualif si on change de valeur à "assurance contactée"
  • Mise à jour de SignalementQualificationUpdater pour fusionner les fonctions addQualificationScore0 et addQualificationScore1To10 et mettre à jour la règle d'apparition de la qualif Assurantielle
  • Mise à jour des tests

Pré-requis

Tests

  • Faire un signalement avec seulement le désordre Le logement est humide et a des traces de moisissures et a Fuite ou dégâts des eaux choisir 'oui' (pour 1 ou plusieurs pièces
  • Valider le signalement (qui doit avoir un score inférieur à 10
  • Modifier le bloc procédures et démarches, "assurance contactée", si l'assurance est contactée, la qualf assurantielle n'apparait pas, sinon elle apparait

@emilschn
Copy link
Collaborator

Quand je crée le signalement, j'ai 4 situations suspectées : Danger occupant, Non décence, RSD, Assurantiel.
Je passe "Contact assurance" de Non à Oui.
Il me reste Non décence et RSD.
Je passe "Contact assurance" de Oui à Non.
Il me reste Non décence, RSD et Assurantiel.

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.

Chose étrange dans le test. Peut-être volontaire ? Mais bizarre.

src/Manager/SignalementManager.php Outdated Show resolved Hide resolved
@numew
Copy link
Collaborator

numew commented Jan 30, 2024

Je viens de voir un autre soucis, si mon signalement à la qualif "Suroccupation" quand je modifie la valeur assurance ca la fait sauter (même problème que celui d'Emilien à première vue)

@hmeneuvrier hmeneuvrier changed the base branch from feature/2073-bo-signalement-update-desordre-qualifications to develop January 30, 2024 12:30
@hmeneuvrier
Copy link
Collaborator Author

hmeneuvrier commented Jan 30, 2024

@emilschn @numew je reproduis facilement pour suroccupation, je regarde çà, ok j'ai compris ! ça vient de la branche précédente, la qualification existe mais en ARCHIVED, et le status n'est pas remis

@hmeneuvrier
Copy link
Collaborator Author

@emilschn @numew @sfinx13
c'est corrigé

Copy link

sonarcloud bot commented Jan 30, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

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

See analysis details on SonarCloud

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.

C'est OK

Copy link
Collaborator

@sfinx13 sfinx13 left a comment

Choose a reason for hiding this comment

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

Test OK

@sfinx13 sfinx13 merged commit c45be41 into develop Jan 30, 2024
3 checks passed
@hmeneuvrier hmeneuvrier deleted the bugfix/2150-algo-modifier-la-partie-assurantiel 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