-
Notifications
You must be signed in to change notification settings - Fork 26
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
Outgoing email logging improvements #4895
Comments
and
seem to be contradicting each other. In the code I can find usages of |
registration emails are tagged, confirmation emails are not but should be, sorry for the confusion! |
It is rather a cosign request email, since the cosign confirmation itself is not much different from the "normal" confirmation email.
This test is not exclusive to appointments, so it makes more sense to have it in the submission app
It is rather a cosign request email, since the cosign confirmation itself is not much different from the "normal" confirmation email.
…ail-logging-improvements [#4895] outgoing email logging improvements
We support sending custom email headers in all emails sent by Open Forms. Currently, this is only used for registration emails. See
open-forms/src/openforms/emails/constants.py
Line 4 in dd38e63
While debugging a production environment to figure out what type of emails were sent, I had to search on the content of the email which is fragile because different forms can use completely different templates rather than a single global configuration. This would've been easier if the registration emails would have been properly tagged with the custom headers so I could filter on this header instead (I was able to exclude registration emails this way).
Additionally, there's a typo in the content type for the
Submission
model - the app label is singular instead of plural.Tasks
The text was updated successfully, but these errors were encountered: