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

[Esabora] Envoi des nouveaux désordres lors du push vers Esabora #2287

Merged
merged 3 commits into from
Feb 27, 2024

Conversation

emilschn
Copy link
Collaborator

Ticket

#2283

Description

Les désordres des signalements du nouveau formulaire n'étaient pas envoyés, depuis le changement de structure.

Tests

  • Créer un signalement dans le 13 et affecter à SCHS et ARS
  • Vérifier en BDD le json envoyé dans la requête (table job_event)

@hmeneuvrier
Copy link
Collaborator

hmeneuvrier commented Feb 26, 2024

Mon signalement a 2 désordres : WC et cuisine dans la même pièce, et des rongeurs :
image
Voici le message pour SISH :
"signalementProblemes":"Probl\u00e8me de structure \/ \u00e9quipement\nIl y a des rongeurs (rats, souris...) "

et pour SCHS :
"dossierCommentaire":"Points signal\u00e9s:\n\nProbl\u00e8me de structure \/ \u00e9quipement\nIl y a des rongeurs (rats, souris...) OUI\nAdultes: Adulte(s)\n0 Enfant(s)\nPartenaire 13-06 ESABORA ARS => En attente..."

n'ayant pas le label de la précision liée, on ne comprend pas quel est le premier désordre.

Copy link

sonarcloud bot commented Feb 26, 2024

Quality Gate Passed Quality Gate passed

Issues
4 New issues

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

See analysis details on SonarCloud

@emilschn
Copy link
Collaborator Author

@hmeneuvrier @numew vos retours sont traités

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.

Test OK et relecture OK.
Juste une réflexion, mais qu'on peut se garder pour plus tard je pense

$commentairePrecision = '';
foreach ($listPrecision as $desordrePrecision) {
if ('' != $desordrePrecision->getLabel()) {
$commentairePrecision .= $desordrePrecision->getLabel().' ; ';
Copy link
Collaborator

Choose a reason for hiding this comment

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

Il y a certaines précisions dont le label comporte du html pour l'affichage sur la fiche signalement.
Il faudra voir si ça ne pose pas de souci côté esabora.
A part ça, pour moi c'est ok.

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.

Test et relecture OK

@numew numew merged commit a32b980 into develop Feb 27, 2024
3 checks passed
@emilschn emilschn deleted the fix/2283-esabora-desordres branch February 29, 2024 13:48
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