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

[Back - Signalement] Il manque très souvent la ville du propriétaire #2419

Merged
merged 2 commits into from
Apr 5, 2024

Conversation

numew
Copy link
Collaborator

@numew numew commented Apr 5, 2024

Ticket

#2413

Description

  • Correction du nom de la propriété concernant la ville du proprio de SignalementDraftRequest.php
  • Utilisation de cette propriété dans le SignalementBuilder.php
  • Ajout d'une migration permettant de récupérer les ville des propriétaire dans le draft (pour les signalement en provenance d'un draft ayant un code postal mais pas de ville d'enregistré pour le proprio)
  • Ajout de tests sur les infos proprio

Tests

  • Ajouter un signalement pour un (ou plusieurs) de ces profil "TIERS_PRO"/"LOCATAIRE"/"TIERS_PARTICULIER"/"SERVICE_SECOURS" et vérifier que la ville propriétaire s'affiche bien en BO
  • Mettre cette ville à NULL à la main en DB et jouer make execute-migration name=Version20240405080750 direction=up puis vérifier que la ville est revenu

@hmeneuvrier
Copy link
Collaborator

Retour de test.
Enregistrement des signalements OK

Migrations, il y a des guillemets en trop

image

Copy link

sonarcloud bot commented Apr 5, 2024

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

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

See analysis details on SonarCloud

@numew
Copy link
Collaborator Author

numew commented Apr 5, 2024

Retour de test. Enregistrement des signalements OK

Migrations, il y a des guillemets en trop

image

C'est corrigé merci

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

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

Tout bon pour moi

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.

Lecture et test OK

@sfinx13 sfinx13 merged commit 3d9f8b8 into main Apr 5, 2024
3 checks passed
@hmeneuvrier hmeneuvrier deleted the hotfix/2413-ville-proprio branch April 11, 2024 13:16
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