Closed Benoit-MINT closed 3 weeks ago
Attention: Patch coverage is 66.66667%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 81.02%. Comparing base (
069cb04
) to head (ee35dba
). Report is 16 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
app/mailers/notification_mailer.rb | 75.00% | 1 Missing :warning: |
app/models/procedure.rb | 50.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
En plus des petits feedbacks ci-dessous, je ne vois pas bien ce qui vérifie cette condition évoquée dans ta PR : pour une démarche auto-instruction, au mail de dépôt uniquement (à moins que l'email en construction soit quand même envoyé ? ça me paraît étrange)
C'est bien exact, pour une démarche en auto-instruction, il y a bien 2 mails envoyés successivement, (i) dépôt/construction et (ii) instruction. J'étais parti du principe que c'était un comportement voulu d'où la "simple" logique utilisée dans ce cas qui évite donc l'effet de doublon.
En réponse à l'issue https://github.com/demarches-simplifiees/demarches-simplifiees.fr/issues/10118 :
Concernant l'apparition dans le mail :