[wp-trac] [WordPress Trac] #44501: Change "From" email address for GDPR request confirmation email
WordPress Trac
noreply at wordpress.org
Tue Feb 4 08:00:56 UTC 2020
#44501: Change "From" email address for GDPR request confirmation email
-------------------------------------------------+-------------------------
Reporter: zaffarn | Owner: xkon
Type: enhancement | Status: assigned
Priority: normal | Milestone: 5.4
Component: Privacy | Version: 4.9.6
Severity: normal | Resolution:
Keywords: has-patch needs-dev-note has-unit- | Focuses:
tests needs-docs commit |
-------------------------------------------------+-------------------------
Changes (by garrett-eclipse):
* keywords: has-patch needs-testing needs-dev-note has-unit-tests needs-
docs => has-patch needs-dev-note has-unit-tests needs-docs commit
Comment:
Thanks @xkon this worked really nicely, some very minor tweaks in
[https://core.trac.wordpress.org/attachment/ticket/44501/44501.4.diff
44501.4.diff]
1. In the `user_request_confirmed_email_headers` docblock corrected the
mention of `$email_text` as it's `$content` that's used. And updated the
$email_data block to match the expected for that email.
2. Updated the description for `user_erasure_complete_email_headers`, and
the `$email_data` contents to match the email.
3. Updated `user_request_action_email_headers` to swap `$email_text` for
`$content`.
Unit tests worked nicely and all my tests cleared (I appreciate the
[https://core.trac.wordpress.org/ticket/44501#comment:6 testing code] made
things simple)
Marking for final review.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/44501#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list