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] Créer des suivis automatiques lors de la modification d'un signalement #2140

Merged

Conversation

hmeneuvrier
Copy link
Collaborator

@hmeneuvrier hmeneuvrier commented Jan 22, 2024

Ticket

#2054

Description

Ajout de suivis automatiques lorsqu'on modifie les différents blocs d'un signalement

Déclencheur Auteur Contenu Partagé à l'usager
Edition de l'adresse Nom de l'agent + nom du partenaire L'adresse du logement a été modifiée par {{nom partenaire}} Non
Edition des coordonnées occupant Nom de l'agent + nom du partenaire Les coordonnées du foyer ont été modifiées par {{nom partenaire}} Non
Edition des coordonnées tiers déclarant Nom de l'agent + nom du partenaire Les coordonnées du tiers déclarant ont été modifiées par {{nom partenaire}} Non
Edition des coordonnées bailleur Nom de l'agent + nom du partenaire Les coordonnées du bailleur ont été modifiées par {{nom partenaire}} Non
Edition des info logement Nom de l'agent + nom du partenaire Les informations sur le logement ont été modifiées par {{nom partenaire}} Non
Edition de la composition du logement Nom de l'agent + nom du partenaire La composition du logement a été modifiée par {{nom partenaire}} Non
Edition de la situation du foyer Nom de l'agent + nom du partenaire La situation du foyer a été modifiée par {{nom partenaire}} Non
Edition de la procédure et démarches Nom de l'agent + nom du partenaire Les procédures et démarches ont été modifiées par {{nom partenaire}} Non

Légères modifications lors des suivis automatiques d'ajout/suppression de documents et photos (le gros du boulot sera à faire lors du traitement du ticket #947 )

Changements apportés

  • Modification sur SignalementEditController
  • Modification sur SignalementFileController

Pré-requis

Tests

  • Modifier un signalement, et vérifier les suivis créés

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, à l'exception d'une petite remarque (en double)

src/Controller/Back/SignalementEditController.php Outdated Show resolved Hide resolved
src/Controller/Back/SignalementFileController.php Outdated Show resolved Hide resolved
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.

Un commentaire, pas encore testé

@hmeneuvrier hmeneuvrier force-pushed the feature/2054-bo-signalement-modification-suivis-automatiques branch from 21555fa to 48f3f47 Compare January 26, 2024 13:04
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.

Ok et rappeler à Arnaud que ça déclenche des mail de notification

@emilschn
Copy link
Collaborator

Tests et relecture ok.
Mais il faudrait voir si c'est ok d'envoyer un mail à chaque fois, ça me semble beaucoup, non ?

@numew
Copy link
Collaborator

numew commented Jan 29, 2024

Tests et relecture ok. Mais il faudrait voir si c'est ok d'envoyer un mail à chaque fois, ça me semble beaucoup, non ?

Si on garde les notif mail, je pense qu'on peux déjà contrôler si il y'a bien une modification effective (en checkant si doctrine fait une requete update) en l'état si on soumet les formulaires sans rien modifier ca notifie également

@hmeneuvrier
Copy link
Collaborator Author

Tests et relecture ok. Mais il faudrait voir si c'est ok d'envoyer un mail à chaque fois, ça me semble beaucoup, non ?

Si on garde les notif mail, je pense qu'on peux déjà contrôler si il y'a bien une modification effective (en checkant si doctrine fait une requete update) en l'état si on soumet les formulaires sans rien modifier ca notifie également

oui c'est une bonne idée !

@sfinx13
Copy link
Collaborator

sfinx13 commented Jan 30, 2024

oui c'est une bonne idée !

Je mets en draft la PR du coup, faudra faire une nouvelle revue

@sfinx13 sfinx13 marked this pull request as draft January 30, 2024 09:58
@hmeneuvrier hmeneuvrier force-pushed the feature/2054-bo-signalement-modification-suivis-automatiques branch from 1ba7ae8 to 11ee645 Compare January 30, 2024 13:45
@hmeneuvrier hmeneuvrier marked this pull request as ready for review January 30, 2024 13:45
@hmeneuvrier hmeneuvrier force-pushed the feature/2054-bo-signalement-modification-suivis-automatiques branch from 0f8ae02 to e4c9a27 Compare February 1, 2024 16:56
Copy link

sonarcloud bot commented Feb 1, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

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

See analysis details on SonarCloud

@numew numew merged commit 1e8a92f into develop Feb 2, 2024
3 checks passed
@sfinx13 sfinx13 deleted the feature/2054-bo-signalement-modification-suivis-automatiques 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

4 participants