Add log when marked for deletion email is sent#609
Add log when marked for deletion email is sent#609Rikukar wants to merge 3 commits intoTampereHacklab:masterfrom
Conversation
| "%(value)s ei ole kelvollinen Matrix id. Sen tulee olla muodossa @käyttäjä:" | ||
| "esimerkki.org" | ||
| "%(value)s ei ole kelvollinen Matrix id. Sen tulee olla muodossa " | ||
| "@käyttäjä:esimerkki.org" |
There was a problem hiding this comment.
Revert this translation change. Adding the log didn't change translations so this should not be changed here
This reverts commit fb6266e.
There was a problem hiding this comment.
Pull request overview
This PR adds activity logging when a "marked for deletion" email is sent to users. This helps track when users are notified about their account deletion through the user's activity log.
- Adds a call to
instance.log()after the email is sent in thehandle_marked_for_deletionsignal handler
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| send_mail(subject, plaintext_content, from_email, to) | ||
|
|
||
| # Add to activity log | ||
| instance.log("Marked for deletion e-mail sent") |
There was a problem hiding this comment.
The term "e-mail" should be "email" for consistency with the rest of the codebase. Throughout the file, "email" (without hyphen) is used consistently in function names (e.g., send_reset_password_email, send_application_received_email) and log messages (e.g., "sending welcome email", "sending bye bye email").
| instance.log("Marked for deletion e-mail sent") | |
| instance.log("Marked for deletion email sent") |
Closes #535