Utiliser l’adresse de réponse magique comme adresse d’envoi des emails de RDV adressés aux usagers #4795
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 leUsers::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 lemail.from
s’il est absent plutôt qu’un simple défaut. Il faudrait le contourner par unUsers::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 overridantdefault_from
on s’économise l’appel à cette méthode.Captures d’écran
J’ai ouvert mon gmail spécialement pour l’occasion
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?