[wp-trac] [WordPress Trac] #43435: Add settings screen for creating a privacy policy

WordPress Trac noreply at wordpress.org
Wed Mar 7 19:35:19 UTC 2018


#43435: Add settings screen for creating a privacy policy
------------------------------+------------------
 Reporter:  azaozz            |       Owner:
     Type:  enhancement       |      Status:  new
 Priority:  normal            |   Milestone:  5.0
Component:  General           |     Version:
 Severity:  normal            |  Resolution:
 Keywords:  gdpr needs-patch  |     Focuses:
------------------------------+------------------

Comment (by allendav):

 @xkon : If we don't specify the slug, wp_insert_post will work anyways and
 manage/version any slug - I recommend removing it

 A few other comments on the patch:

 1) Lets use `__( 'Privacy Policy' )` for the post_title argument to
 wp_insert_post

 2) Do we really need the incremental complexity of AJAX for this part? How
 about just another form+button that posts a create-privacy-page action in
 a similar manner to how tools-privacy.php (in this patch) currently
 handles the set-privacy-page action? We can then have add_settings_error
 emit something like "Privacy policy page created successfully" similar to
 how the updated action does. If we get rid of the AJAX, the little bit of
 CSS can go too.

 AJAX will be needed later, certainly, for things like exporting or
 removing a user's personal data, but let's not bother with it for this,
 ok?

 3) Let's keep the view/edit link (if a page is selected) but it needs to
 be localized. Let's keep the jQuery in the file but just have it hide the
 view/edit link if the user changes the selected page in the dropdown (i.e.
 1) to avoid confusion by having the link no longer link to the page
 displayed in the SELECT and 2) to avoid people not saving their changes


 Mind if I submit a 43435.4 with those changes?

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


More information about the wp-trac mailing list