#44030 closed enhancement (fixed)
Remove unnecessary `This email has been sent to ###EMAIL###` from privacy emails.
Reported by: | iandunn | Owned by: | ianbelanger |
---|---|---|---|
Milestone: | 4.9.7 | Priority: | low |
Severity: | normal | Version: | 4.9.6 |
Component: | Privacy | Keywords: | gdpr good-first-bug has-patch commit |
Focuses: | Cc: |
Description
Several privacy related emails contain a string in the body saying, This email has been sent to ###EMAIL###
. It appears that this was copied from the emails that get sent when an email address changes, without considering if it made sense in the new context.
I think that string is appropriate in the original use cases, because explicitly calling out the receiving address can help avoid confusion, since the address is changing.
The string is not used in any other emails in Core, though, except for some of the new privacy ones from 4.9.6. I think the string is unnecessary in the new context, and it's best to remove it.
The recipient address is already obvious enough from the To
header, and removing unnecessary content will make the messages shorter, and maybe even avoid some suspicion from spam filters, which sometimes penalize seemingly innocent text, especially if it contains URLs or email addresses.
Attachments (2)
Change History (11)
This ticket was mentioned in Slack in #core by desrosj. View the logs.
6 years ago
#4
@
6 years ago
- Milestone changed from 4.9.7 to Future Release
Moving gdpr
tickets that are not bugs to Future Release
until the next steps can be properly evaluated.
#7
@
6 years ago
- Keywords commit added; needs-testing removed
- Milestone changed from Future Release to 4.9.7
- Owner set to ianbelanger
- Status changed from new to assigned
- Version changed from trunk to 4.9.6
Assigning to mark good first bug as claimed.
This tested well for me. I am uploading one small change. Because the user's email is removed from the text of the email, the note explaining the ###EMAIL###
placeholder can be removed from the translation context and the inline documentation for the filters on that content.
Removed 2 instances of the unnecessary email text