[wp-trac] [WordPress Trac] #43970: Add request type to the confirmation confirmation page - GDPR

WordPress Trac noreply at wordpress.org
Thu May 10 19:44:14 UTC 2018


#43970: Add request type to the confirmation confirmation page - GDPR
------------------------------------------+-----------------------------
 Reporter:  garrett-eclipse               |       Owner:  desrosj
     Type:  defect (bug)                  |      Status:  assigned
 Priority:  normal                        |   Milestone:  4.9.6
Component:  General                       |     Version:  trunk
 Severity:  normal                        |  Resolution:
 Keywords:  gdpr has-patch needs-testing  |     Focuses:  administration
------------------------------------------+-----------------------------
Changes (by desrosj):

 * keywords:  gdpr has-patch needs-refresh => gdpr has-patch needs-testing


Comment:

 [attachment:43970.2.diff] has a small handful of improvements.

 - Updates the confirmation notices to match the suggestions on #44029
 - Move the `user_request_action_confirmed_message` filter into the
 `_wp_privacy_account_request_confirmed_message()` function. The function
 was filtering the exact same text in the `wp-login.php` file, so was
 somewhat superfluous. The filter should still exist, though.
 - Added an inline docblock for the `user_request_action_confirmed_message`
 filter.
 - Added missing parameter documentation in the
 `_wp_privacy_account_request_confirmed_message()` function.

 @melchoyce I used the same green color that the admin uses for success
 messages. It seemed a big darker than in your mockup from #44029, though.
 Please check that you are ok with this color, or just send me a different
 hex value.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/43970#comment:12>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list