[wp-trac] [WordPress Trac] #43921: Include community-events-location user meta value in Personal Data Export
WordPress Trac
noreply at wordpress.org
Wed Jan 29 08:27:11 UTC 2020
#43921: Include community-events-location user meta value in Personal Data Export
-------------------------------------------------+-------------------------
Reporter: coreymckrill | Owner: garrett-
| eclipse
Type: enhancement | Status: accepted
Priority: normal | Milestone: 5.4
Component: Privacy | Version: 4.9.6
Severity: normal | Resolution:
Keywords: has-patch needs-testing has- | Focuses:
screenshots has-unit-tests |
-------------------------------------------------+-------------------------
Changes (by garrett-eclipse):
* keywords: has-patch 2nd-opinion => has-patch needs-testing has-
screenshots has-unit-tests
* status: reviewing => accepted
* milestone: Future Release => 5.4
Comment:
Thanks again for the patch @coreymckrill sorry for the delay getting back
around to this.
I've tested the 43921.diff patch successfully but found when the data
contains multiple location attributes it can be convoluted on one line as
well the description of 'Users profile data' leads one to believe that all
information listed is available from the profile. Due to this I felt it
would benefit to be split out into it's own section.
I've uploaded
[https://core.trac.wordpress.org/attachment/ticket/43921/43921.2.diff
43921.2.diff] to switch to a dedicated group and introduce unit tests
specific to this. Screenshots uploaded.
*Note: To my knowledge, there will only ever be one community event
location per user? If this is incorrect we can loop the data to present it
similar to when there's multiple comments.
Please review and let me know what you think? I feel we can probably land
this in 5.4 if we're able to get some testing done here.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/43921#comment:11>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list