[OKR][Emails] Modifier les objets des emails partenaires

[sfinx13]

Ticket

#2093
#1365
#2091

Description

• Mise à jour des objets de mail pour les partenaires
• Ne pas ajouter de suivi de première affectation si visite déjà planifiée ou effectuée
• Notifier pour les visites qui doivent être 15 jours après qu'une affectation soit affectée

Changements apportés

• Mise à jour des objets de mail dans les services dédiées
• Ajout de fixtures et de tests pour avoir tous les cas de tests d'email
• Ajout d'un service dédié pour condition l'envoi un message sur le premier affectation
• Activation de la variable d'environnement pour les tests et CI

Pré-requis

Mise à jour de la variable d'environnement dans votre .env.local

FEATURE_ASK_VISITE_ENABLE=1

Tests

Objet actuel	Nouvel objet
Nouveau suivi sur le signalement 2023-123	[2023-123 - Dupont] Nouveau suivi
Un nouveau signalement vous attend	[67980] Un nouveau signalement vous attend
SCHS a terminé son intervention sur 2023-123	[2023-123 - Dupont] SCHS a terminé son intervention
Le signalement 2023-123 a été cloturé	[2023-123 - Dupont] Clôture du signalement
2023-123 Visite du logement prévue	[2023-123 - Dupont] Visite programmée
2023-123 Veuillez renseigner la conclusion de visite	[2023-123 - Dupont] Conclusion de visite à renseigner
2023-123 Veuillez renseigner une date de visite	[2023-123 - Dupont] Date de visite à renseigner
2023-123 Conclusion de visite disponible	[2023-123 - Dupont] Conclusion de visite disponible
2023-123 Visite non effectuée	[2023-123 - Dupont] Visite non effectuée

• ([Emails] Modifier les objets des emails partenaires #2093) Lancer les tests et vérifier (via mailcatcher) que les objets de mail de la liste ci-dessus ont bien été mis à jour
  

• ([Emails usager] Vérifier qu'il n'y a pas de mail de visite déjà envoyé  #1365) Dans le cas où un email visite a été envoyé : ne pas créer de suivi "première affectation acceptée"

1. Connecter vous en tant que RT
2. Créer une visite passé ou/et futur => Un email visite est envoyé à l'usager
3. Affecter un partenaire
4. Connecter vous en tant qu'utilisateur partenaire
5. Accepter le signalement => Suivi de première affectation donc pas de mail de notif usager

• ([Emails usager] Vérifier qu'il n'y a pas de mail de visite déjà envoyé  #1365) Dans le cas où aucun email visite n'a été envoyé : créer un suivi "première affectation acceptée"

1. Connecter vous en tant que RT
2. Affecter un partenaire
3. Connecter vous en tant qu'utilisateur partenaire
4. Accepter le signalement => Suivi de première affectation donc email de notif usager

• ([Emails] Activer l'email "date de visite à renseigner" #2091) En tant qu'utilisateur partenaire, accepter une affectation et modifier la date d'acceptation dans la base de données, de manière à la mettre à jour 15 jours avant la date du jour et exécuter la commande make console app=notify-visits
  Le partenaire doit recevoir le mail Date de visite à renseigner
  

[emilschn]

Un retour qui se répète + une demande d'explication sur un nom de méthode :)
Je n'ai pas encore testé

[hmeneuvrier]

Pour le mail "Un nouveau signalement vous attend", les partenaires ont l'objet "sur Histologe", et pas les RT (c'est pas grave, mais je ne sais pas si c'est voulu)

[hmeneuvrier]

Je croyais que ce mail devenait "Visite programmée"

[hmeneuvrier]

Emails usager] Vérifier qu'il n'y a pas de mail de visite déjà envoyé #1365) : de ce que je lis du ticket, le mail ne doit pas être envoyé dans ce cas, mais je ne vois pas que le suivi ne doit pas être créé.

EDIT : je viens de voir ton commentaire, je ne sais pas quelle a été la réponse de Mathilde, mais en tous cas ça fonctionne, si une vite est créée avant la première acceptation d'affectation, alors il n'y a pas de suivi créé, et du coup pas de mail envoyé à l'usager

[hmeneuvrier]

Je viens de faire le dernier test, on reçois bien le mail, mais il y a quelques trucs bizarres :

• dans le sujet on a [2023-20 - Colas] et je ne sais pas à quoi correspond "Colas", cela devrait être le nom de l'occupant et ce n'est pas le cas, il s'appelle bertrand Da Costa
• on a un bouton "accéder à mon signalement" qui envoie faire la page signalement usager, alors que c'est le partenaire et que ça devrait être "accéder au signalement" avec un lien vers la page BO

[hmeneuvrier]

AH !!

il y a 2 signalements 2023-20 pour ce partenaire... super bizarre...

ok c'est dans les fixtures qu'il y en a 2, avec même référence, même adresse mais uuid différent...
Et je me demande s'il n'y a pas un encroit où on cherche par référence/territoire et du coup ça n'a pas pris le bon signalement.

[sfinx13]

Pour le mail "Un nouveau signalement vous attend", les partenaires ont l'objet "sur Histologe", et pas les RT (c'est pas grave, mais je ne sais pas si c'est voulu)

C'était déjà comme ça mais c'est redondant avec l'objet je vais enlever

[sfinx13]

Emails usager] Vérifier qu'il n'y a pas de mail de visite déjà envoyé #1365) : de ce que je lis du ticket, le mail ne doit pas être envoyé dans ce cas, mais je ne vois pas que le suivi ne doit pas être créé. EDIT : je viens de voir ton commentaire, je ne sais pas quelle a été la réponse de Mathilde, mais en tous cas ça fonctionne, si une vite est créée avant la première acceptation d'affectation, alors il n'y a pas de suivi créé, et du coup pas de mail envoyé à l'usager

C'est ça et vu avec Mathilde en privée vendredi dernier (j'ai confirmé aussi dans le ticket :-) )

[sfinx13]

• 2023-20

Je vais corriger la fixture @hmeneuvrier

[emilschn]

Ma relecture est ok, mes tests aussi, je pense avoir vérifié tous les mails listés.

Petite question : est-ce que ce ne serait pas plus simple d'avoir une méthode dans l'entité Signalement qui se charge de faire
$signalement->getNomOccupant() ?? $signalement->getNomDeclarant()
plutôt que le répéter plein de fois ?

[sfinx13]

Ma relecture est ok, mes tests aussi, je pense avoir vérifié tous les mails listés.

Petite question : est-ce que ce ne serait pas plus simple d'avoir une méthode dans l'entité Signalement qui se charge de faire $signalement->getNomOccupant() ?? $signalement->getNomDeclarant() plutôt que le répéter plein de fois ?

Bonne idée c'est fait @emilschn

[sonarcloud]

Quality Gate failed

Failed conditions

28.7% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

[hmeneuvrier]

Relecture et re-tests OK

[emilschn]

Lecture et tests validés :)