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

[Signalement - bail-dpe] Ajouter bouton upload état des lieux #2159

Merged
merged 1 commit into from
Jan 31, 2024

Conversation

hmeneuvrier
Copy link
Collaborator

@hmeneuvrier hmeneuvrier commented Jan 30, 2024

Ticket

#2139

Description

De la même manière que pour le bail ou le DPE, ajouter un bouton d'upload pour l'état des lieux quand on coche "Oui" à la question Avez-vous fait un état des lieux quand vous avez emménagé ? (ou Avez-vous réalisé un état des lieux à l'entrée des locataires ? pour les bailleurs)

Changements apportés

  • Modification des json de questions
  • Modification d'une fixture
  • Ajout d'un test
  • Modification du SignalementDocumentTypeMapper

Pré-requis

Tests

  • LFaire un signalement en tant que locataire, vérifier qu'on peut ajouter un fichier pour l'état des lieux. Ajouter un fichier et finir le signalement. Dans le BO, vérifier que le document de l'état des lieux est accessible. Dans la bdd, vérifier qu'il est bien typé
  • Idem en tant que bailleur

Copy link

sonarcloud bot commented Jan 30, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

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

See analysis details on SonarCloud

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 test et relecture

@sfinx13 sfinx13 merged commit 627c8b4 into develop Jan 31, 2024
3 checks passed
@sfinx13 sfinx13 deleted the feature/2139-signalement-upload-etat-des-lieux branch February 13, 2024 21:09
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

3 participants