#44006 closed enhancement (fixed)
Privacy Policy page should have suffix like other special pages
Reported by: | danieltj | Owned by: | desrosj |
---|---|---|---|
Milestone: | 4.9.8 | Priority: | normal |
Severity: | minor | Version: | 4.9.6 |
Component: | Privacy | Keywords: | has-screenshots has-patch commit |
Focuses: | administration | Cc: |
Description
The Privacy Policy page (when set in the Privacy page within Tools > Privacy
) doesn't have the special page suffix like the Home and Blog pages do. As this page has special options when officially set as The Privacy Policy Page, it needs to have a suffix so people can pick it out easily from the list of pages.
Attachments (6)
Change History (27)
This ticket was mentioned in Slack in #gdpr-compliance by allendav. View the logs.
6 years ago
#4
@
6 years ago
- Component changed from Posts, Post Types to Privacy
Moving to the new Privacy component.
This ticket was mentioned in Slack in #core-privacy by desrosj. View the logs.
6 years ago
#8
@
6 years ago
- Keywords needs-testing added
- Milestone changed from Awaiting Review to 4.9.7
- Version changed from trunk to 4.9.6
This is a great idea.
44006.2.diff has a few minor improvements:
- Remove the first conditional, it was superfluous because the second condition would always catch the first too.
- Added some missing spaces to adhere to the coding standards.
- Changed the text to
Privacy Policy Page
for consistency with the option name.
#9
@
6 years ago
- Keywords needs-testing removed
Tested 44006.2.diff and the patch applies cleanly and works as intended. Screenshot coming.
This ticket was mentioned in Slack in #core-privacy by desrosj. View the logs.
6 years ago
#11
@
6 years ago
As discussed in today's Privacy component bug scrub, the text seems a bit repetitive when the page's name is "Privacy Policy". But not sure that there is anything we can do about this. The page title is not guaranteed to be "Privacy Policy" (a user could change it any time they want).
#12
@
6 years ago
Just tested the latest patch and the Privacy Policy link is removed when there is no title for the Privacy Policy page.
Nice work @desrosj
#13
@
6 years ago
- Milestone changed from 4.9.7 to 4.9.8
4.9.7 has been released, moving to next milestone.
This ticket was mentioned in Slack in #core by pbiron. View the logs.
6 years ago
#15
@
6 years ago
Is this ready to land in 4.9.8?
The general feeling during today's bug scrub is that it is ready, but just want to confirm. If so, will the patch need a @since 4.9.8
refresh?
#16
follow-up:
↓ 17
@
6 years ago
This is good to go. My only remaining question is the casing of "page". See the discussion on 44130#comment:7.
#17
in reply to:
↑ 16
@
6 years ago
Replying to desrosj:
This is good to go. My only remaining question is the casing of "page". See the discussion on 44130#comment:7.
I don't have strong feelings one way or the other about 44130#comment:7, but for consistency with the Front Page
and Posts Page
post_states, I'd prefer Privacy Policy Page
for this one.
Privacy Policy page without the special page suffix