[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