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.

Sign up for GitHub

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

Jump to bottom

[Emails] Nouveau mail de demande d'ajout de documents #3291

Merged
merged 9 commits into from
Nov 20, 2024
Merged

[Emails] Nouveau mail de demande d'ajout de documents #3291

merged 9 commits into from
Nov 20, 2024

Conversation

emilschn
Copy link
Collaborator

@emilschn emilschn commented Nov 14, 2024 •
edited
Loading

Ticket

#3221

Description

Lorsqu'un usager crée un signalement, si il n'a pas ajouté de documents dans les parties où c'est nécessaire, on lui renvoie un e-mail avec un petit décalage pour qu'il envoie les documents.

Changements apportés

  • Ajout d'un nouveau type de message pour Messenger
  • Variable d'environnement qui permet d'activer la fonctionnalité : FEATURE_CHECK_NEW_SIGNALEMENT_FILES
  • Variable d'environnement qui permet de paramétrer le délai : DELAY_MIN_CHECK_NEW_SIGNALEMENT_FILES
    • En minutes
    • Par défaut : 30

Tests

  • TNR : si on n'active pas la variable d'environnement, rien ne change lorsqu'on crée un signalement (entre message normal et demande de doc NDE)
  • Si on active, on ne reçoit plus le message de doc NDE, on reçoit toujours le message de signalement bien reçu
  • Si on a transmis tous les docs (ou si ils n'étaient pas nécessaires), on ne reçoit pas de message supplémentaire
  • Si on n'a pas transmis tous les docs (bail / DPE / Etat des lieux OU un des docs associé à désordre qui nécessite un doc), on reçoit le message avec le détail des docs oubliés

@emilschn emilschn marked this pull request as draft November 14, 2024 09:47
@emilschn emilschn changed the title [WIP] [Emails] Nouveau mail de demande d'ajout de documents [Emails] Nouveau mail de demande d'ajout de documents Nov 14, 2024
@emilschn emilschn marked this pull request as ready for review November 14, 2024 11:33
numew
numew reviewed Nov 14, 2024
View reviewed changes
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.

quelques questions et un test KO :
Je renseigne un signalement en mettant une photo sur la partie nuisible, et on me demande des photo sur ce désordre
Screenshot 2024-11-14 at 16-07-58 Suivre mon signalement #2024-8 - Histologe
Screenshot 2024-11-14 at 16-07-36 Signalement - Histologe

src/Messenger/MessageHandler/NewSignalementCheckFileMessageHandler.php

public function __invoke(NewSignalementCheckFileMessage $newSignalementCheckFileMessage): void
{
$this->logger->info('Start handling NewSignalementCheckFileMessage', [
Copy link
Collaborator

Choose a reason for hiding this comment

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

c’était pour test ou tu veux garder ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

c'était le cas sur un autre message, j'ai juste copié en me disant que ça permettrait de débug au début. Rien d'obligatoire.

src/Messenger/MessageHandler/NewSignalementCheckFileMessageHandler.php Outdated
$this->suivi = $this->createSuivi($signalement, $documents, $desordres);
}

$this->logger->info('NewSignalementCheckFileMessage handled successfully', [
Copy link
Collaborator

Choose a reason for hiding this comment

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

même question, on garde ?

@emilschn emilschn force-pushed the feature/3221-check-new-signalements-docs branch from a387faa to ad511d9 Compare November 14, 2024 15:58
sfinx13
sfinx13 reviewed Nov 14, 2024
View reviewed changes
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.

Pas encore testé

numew
numew reviewed Nov 18, 2024
View reviewed changes
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.

J'ai essayé de faire le tour, je pense qu'il manque ce désordre à ajouter à la liste :
Screenshot 2024-11-18 at 14-54-41 Signaler un problème de logement - Histologe

@emilschn emilschn force-pushed the feature/3221-check-new-signalements-docs branch from cf93e15 to 47bca75 Compare November 18, 2024 16:00
@emilschn
Copy link
Collaborator Author

J'ai essayé de faire le tour, je pense qu'il manque ce désordre à ajouter à la liste : Screenshot 2024-11-18 at 14-54-41 Signaler un problème de logement - Histologe

Merci, je suis effectivement passé à côté !

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

sfinx13
sfinx13 approved these changes Nov 19, 2024
View reviewed changes
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

numew
numew approved these changes Nov 20, 2024
View reviewed changes
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.

ok

@numew numew merged commit 592bbf2 into develop Nov 20, 2024
3 checks passed
@hmeneuvrier hmeneuvrier deleted the feature/3221-check-new-signalements-docs branch November 26, 2024 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sfinx13 sfinx13 sfinx13 approved these changes

@numew numew numew approved these changes

@hmeneuvrier hmeneuvrier Awaiting requested review from hmeneuvrier

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@emilschn @sfinx13 @numew