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

[BO - Signalement] Refacto - Affichage des informations de non décence sur la fiche signalement #1013

Merged
merged 5 commits into from
Mar 10, 2023

Conversation

hmeneuvrier
Copy link
Collaborator

Ticket

#932

Description

  • L'ajout d'une partie dédiée à la non décence énergétique dans la fiche signalement
    
  • Ouverture d'une modale pour éditer les informations
    

Spécs : https://github.com/MTES-MCT/histologe/wiki/Gestion-NDE/
Maquette fiche signalement avec infos non décence énergétique

Changements apportés

  • renommer desordres en criticites dans la table signalement_qualification
  • création des signalement_qualification pour les signalements en fixtures
  • création d'un enum QualificationStatus
  • Dans BackSignalementController, récupération de la qualification et envoi des informations à la vue pour afficher la nouvelle partie sur la NDE
  • Création d'une modale pour éditer ces informations
  • Création de BackSignalementQualificationController pour enregistrer les changements
  • Création de SignalementQualificationService pour mettre à jour le status de la qualification en fonction des données

Tests

  • Rejouer les fixtures, et la commande symfony console app:commune-filler et symfony console app:update-partners-communes-nde pour avoir les bonnes données
  • Se connecter en admin, ouvrir le signalement du Puy de Dôme, le valider (2023-8)
  • Vérifier que la partie Non Décence énergétique s'affiche bien, que l'édition des informations et leur enregistrement se passe bien (ne tester que la consommation énergétique pour l'instant, le reste est plus loin, essayer des valeurs sous et au-dessus de 450 pour vérifier que le statut du signalement change selon)
  • Affecter des partenaires avec et sans la compétence Non Décence énergétique (ADIL avec la compétence ALF (OPAH) sans)
  • Se connecter avec ALF (OPAH) (user-63-02@histologe.fr) vérifier qu'on a accès au signalement, mais qu'on ne voit pas la partie non décence énergétique
  • Se connecter avec ADIL (user-63-01@histologe.fr) vérifier qu'on a accès au signalement, et qu'on voit la partie non décence énergétique. Changer la date du DPE avant/après 2023 pour vérifier qu'en 2023 le champ superficie ne s'affiche plus. Changer la date du bail, pour vérifier que si le dernier bail est antérieur à 2023, le signalement n'est plus en non décence énergétique, la partie ne s'affiche plus, et en base il est Archivé.

@sonarcloud
Copy link

sonarcloud bot commented Mar 10, 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 4 Code Smells

No Coverage information No Coverage information
7.1% 7.1% Duplication

@hmeneuvrier hmeneuvrier marked this pull request as ready for review March 10, 2023 09:45
@sfinx13 sfinx13 merged commit 24d8f50 into feature/nde-lot1 Mar 10, 2023
@hmeneuvrier hmeneuvrier deleted the feature/932-refacto branch March 21, 2023 14:56
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