#46056 closed defect (bug) (fixed)
The personal data export email for privacy requests should be in the users locale
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 5.2 | Priority: | normal |
Severity: | normal | Version: | 4.9.6 |
Component: | I18N | Keywords: | has-patch has-unit-tests |
Focuses: | administration, privacy | Cc: |
Description
Hello,
From testing in #44721 we found that the second email sent to the user (the fulfillment email) is not in their locale but it should be.
Example from @birgire - https://core.trac.wordpress.org/attachment/ticket/44721/user-fullfilment-email.jpg
Attachments (8)
Change History (19)
#1
@
6 years ago
- Version 4.9.6 deleted
While waiting for Berlin (and the rest of the roadmap), a ticket like this which i think we can all agree - is a good idea, could be expedited.
#3
@
6 years ago
- Keywords has-patch needs-testing needs-unit-tests added; needs-patch removed
- Milestone changed from Future Release to 5.2
- Owner set to garrett-eclipse
- Status changed from new to accepted
- Version set to 4.9.6
Mimicking @birgire's successful patch (44721-2.diff) from #44721 I've applied the same logic into the wp_privacy_send_personal_data_export_email
function found in file.php
to create patch 46056.diff.
#4
@
6 years ago
- Summary changed from The fulfillment emails for export and erasure privacy requests should be in the users locale to The fulfillment email for export privacy requests should be in the users locale
#5
@
6 years ago
- Keywords has-unit-tests added; needs-unit-tests removed
- Summary changed from The fulfillment email for export privacy requests should be in the users locale to The personal data export email for privacy requests should be in the users locale
Uploaded 46056.2.diff to provide unit tests which follow the same as was laid out in #44721. Attached .mo files should be placed in the tests/phpunit/data/languages folder as they couldn't be included in the svn diff, their .po counterparts are there.
*This will require a refresh once #44721 is committed
#6
@
6 years ago
- Component changed from Privacy to I18N
- Focuses privacy added
Updated component/focus to match it's sister ticket #44721
This ticket was mentioned in Slack in #core-privacy by garrett-eclipse. View the logs.
6 years ago
@
6 years ago
Refresh to account for #44721 being committed - Changed are isolated to .po test files. .mo versions coming shortly
#8
@
6 years ago
Refreshed the .po and .mo files due to #44721 being committed to trunk 🎉.
Note to testers and committers: For successful unit tests place the de_DE.2.mo and es_ES.2.mo into the .../tests/phpunit/data/languages
folder as de_DE.mo and es_ES.mo respectively.
*Sorry can't include in svn diff due to binary files being excluded from diffs.
Export Fulfillment email that should be in user locale