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

Utiliser l’adresse de réponse magique comme adresse d’envoi des emails de RDV adressés aux usagers #4795

Merged
merged 2 commits into from
Nov 12, 2024

Conversation

adipasquale
Copy link
Contributor

@adipasquale adipasquale commented Nov 12, 2024

Closes #4682

Review app : https://demo-rdv-solidarites-pr4795.osc-secnum-fr1.scalingo.io/

Contexte

cf ticket #4682 piste A

Solution

J’override la méthode default_from appelée par l’ApplicationMailer dont le Users::RdvMailer hérite.

Ce n’est pas le plus facilement lisible mais c’est le plus simple à mettre en place car il y a un étrange ApplicationMailer#after_action qui définit le mail.from s’il est absent plutôt qu’un simple défaut. Il faudrait le contourner par un Users::RdvMailer#before_action sinon.

On veut aussi utiliser la méthode de ApplicationMailer pour afficher le bon nom d’expéditeur selon le domaine. En overridant default_from on s’économise l’appel à cette méthode.

Captures d’écran

J’ai ouvert mon gmail spécialement pour l’occasion

image

On voit que le contact apparaît bleu avant et maintenant gris. Je ne sais pas trop ce que ça symbolise chez GMail, peut-être que le contact est déjà connu ou non?

@adipasquale adipasquale force-pushed the feature/default-from-users-rdv-mailer branch from 9fd8421 to 99b85f7 Compare November 12, 2024 08:00
@adipasquale adipasquale marked this pull request as ready for review November 12, 2024 08:22
Copy link
Contributor

@victormours victormours left a comment

Choose a reason for hiding this comment

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

👍 Nice
Dans l'issue tu demandes si faire ce changement risque de diminuer notre score de réputation, est-ce que tu as plus d'infos là dessus ?

@adipasquale
Copy link
Contributor Author

@victormours non je n’ai pas plus d’infos :/ j’ai cherché des infos sans succès à ce sujet.

@victormours
Copy link
Contributor

@victormours non je n’ai pas plus d’infos :/ j’ai cherché des infos sans succès à ce sujet.

En cherchant un peu de mon côté, j'ai l'impression que les vérification se font surtout au niveau du nom de domaine de l'envoyeur (et de l'adresse IP), donc ça me semble pas trop risqué effectivement. 👍

@adipasquale adipasquale enabled auto-merge (squash) November 12, 2024 14:23
@adipasquale adipasquale merged commit 291a9bf into production Nov 12, 2024
15 of 16 checks passed
@adipasquale adipasquale deleted the feature/default-from-users-rdv-mailer branch November 12, 2024 14:39
adipasquale added a commit that referenced this pull request Nov 13, 2024
…es emails de RDV adressés aux usagers (#4795)"

This reverts commit 291a9bf.
adipasquale added a commit that referenced this pull request Nov 13, 2024
…es emails de RDV adressés aux usagers (#4795)" (#4802)

This reverts commit 291a9bf.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

Diminuer le nombre de tickets usagers Zammad suite à une réponse aux mails transactionnels
2 participants