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

[A supprimer] Formulaire front pour la Non décence énergétique #989

Closed

Conversation

emilschn
Copy link
Collaborator

@emilschn emilschn commented Mar 3, 2023

Ticket

#931

Description

Mise en place de l'affichage et de l'enregistrement du formulaire avec les nouvelles données sur la Non-décence énergétique.
Les détails du parcours se trouvent ici : https://github.com/MTES-MCT/histologe/wiki/Non-Decence---Parcours

Changements apportés

  • Prise en compte des 2 territoires d'expérimentation (63 + 89) lors de la saisie de l'adresse
  • Prise en compte des criticités liées à la NDE
  • Si ces conditions sont réunies, on affiche un nouvel onglet avec un parcours spécifique pour établir les cas de NDE
  • Une nouvelle partie apparait dans le résumé du dernier onglet, si nécessaire
  • Les informations s'enregistrent dans un SignalementQualification

Tests

  • Ajouter un signalement sur un territoire non concerné, et vérifier que rien ne change
  • Ajouter un signalement avec des désordres non-liés, et vérifier que rien ne change
  • Faire 5 tests différents de signalements NDE, vérifier que le parcours est bon, que le résumé et que l'enregistrement correspondent

sfinx13 and others added 4 commits March 6, 2023 09:51
[BO - Dashboard] Correction de l'affichage des affectations des partenaires dans le tableau de bord
@hmeneuvrier
Copy link
Collaborator

@emilschn En me basant sur le parcours, j'ai fait ces 6 tests :

Test 1
date entrée 2023
DPE 2023
900

Test 2
date entrée 2023
DPE non

Test 3
date entrée 2023
DPE ne sait pas

Test 4
date entrée 2023
DPE oui avant 2023

Test 5
date entrée avant 2023
bail renouvelé en 2023
non

Test 6
date entrée avant 2023
bail renouvelé avant 2023

Pour les 5 premiers, j'ai bien le comportement attendu.
Pour le 6è, j'ai bien le texte "Votre logement n'est pas concerné par les changements de législation concernant la consommation énergétique. Cliquez sur le bouton "Continuer" pour compléter votre signalement sur Histologe !" Par contre au récap, j'ai les infos suivantes, je ne sais pas si c'est normal ?
image

@hmeneuvrier
Copy link
Collaborator

Mais quand je l'ouvre dans le BO, je n'ai pas l'info NDE, donc ça c'est ok

@hmeneuvrier
Copy link
Collaborator

Pour mon test 4, j'ai rentré une superficie et une consommation d'énergie, le calcul a été fait, par contre je pense qu'on a un truc à revoir car dans la fiche signalement, au lieu d'avoir 80000 kWh/an et 80 m2, j'ai 1000 kWh/an ce qui est le calcul effectué et rien pour la superficie. A mon avis il faut qu'on se cale, on doit pas utiliser les mêmes infos entre ce que tu enregistres et ce que j'affiche.

image

@hmeneuvrier
Copy link
Collaborator

@emilschn est-ce que c'est normal quand on n'a pas de DPE d'avoir une analyse NDE avérée plutôt que check ?

@hmeneuvrier
Copy link
Collaborator

@emilschn en fait j'en ai aussi un (test 3) sans DPE avec non décence OK

@emilschn
Copy link
Collaborator Author

emilschn commented Mar 7, 2023

@emilschn est-ce que c'est normal quand on n'a pas de DPE d'avoir une analyse NDE avérée plutôt que check ?

@hmeneuvrier Oui, c'est attendu comme ça, bizarrement.

Pour le 6è, j'ai bien le texte "Votre logement n'est pas concerné par les changements de législation concernant la consommation énergétique. Cliquez sur le bouton "Continuer" pour compléter votre signalement sur Histologe !" Par contre au récap, j'ai les infos suivantes, je ne sais pas si c'est normal ?

Non, le récap n'est pas normal, je vais ajuster.

Pour mon test 4, j'ai rentré une superficie et une consommation d'énergie, le calcul a été fait, par contre je pense qu'on a un truc à revoir car dans la fiche signalement, au lieu d'avoir 80000 kWh/an et 80 m2, j'ai 1000 kWh/an ce qui est le calcul effectué et rien pour la superficie. A mon avis il faut qu'on se cale, on doit pas utiliser les mêmes infos entre ce que tu enregistres et ce que j'affiche.

Je n'enregistrais que le calcul jusque là, car on peut saisir juste le calcul. Du coup, il manque quelque chose dans ton formulaire (pour les DPE 2023) pour n'afficher que le calcul. Effectivement, il faut qu'on se mette d'accord.

@sonarcloud
Copy link

sonarcloud bot commented Mar 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 11 Code Smells

No Coverage information No Coverage information
3.3% 3.3% Duplication

@sfinx13
Copy link
Collaborator

sfinx13 commented Mar 8, 2023

@emilschn cette branche n'est pas mergeable dans l'état, beaucoup de fichiers qui ne devrait pas être la (du dev de develop)

@emilschn emilschn changed the title Formulaire front pour la Non décence énergétique [A supprimer] Formulaire front pour la Non décence énergétique Mar 9, 2023
@emilschn emilschn closed this Mar 9, 2023
@emilschn emilschn deleted the feature/931-front-nte-tab branch March 10, 2023 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants