[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