Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Separate "Privacy, search, and services" Edge Settings in regular articles #3254

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

mikehoffms
Copy link
Contributor

@mikehoffms mikehoffms commented Aug 28, 2024

Merge this PR at same time as PR https://github.com/MicrosoftDocs/DocsLegal-pr/pull/1648 (Separate "Privacy, search, and services" Edge Settings into "Privacy and security" & "Search and services") in Legal repo.

Rendered articles for review:

Microsoft Edge > Settings and more > Settings > Privacy, search, and services is possibly getting split into two sections:

  • Privacy and security
  • Search and services

That change impacts the Microsoft Edge Privacy Whitepaper, which mentions the old combined section, and mentions "edge://settings/privacy" 36 times. Two other articles also mention "edge://settings/privacy". Privacy article was moved to Legal repo.

This PR was created by copying changes from the Closed Without Merging PR #3127 (Separate "Privacy, search, and services" into "Privacy and security" & "Search and services") but not for the Privacy article; only for two regular articles.

@mikehoffms mikehoffms added the on-hold The changes are currently on-hold and awaiting further evaluation. label Aug 28, 2024
Copy link
Contributor Author

@mikehoffms mikehoffms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR LGTM, per Writer/Editor review.

Copy link

Learn Build status updates of commit 764af8d:

✅ Validation status: passed

File Status Preview URL Details
microsoft-edge/web-platform/tracking-prevention-images/settings-and-more-icon.png ✅Succeeded View
microsoft-edge/web-platform/tracking-prevention.md ✅Succeeded View
microsoft-edge/webview2/concepts/data-privacy-images/settings-and-more-icon.png ✅Succeeded View
microsoft-edge/webview2/concepts/data-privacy.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

@apryzgoda apryzgoda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Edge Privacy Champ approved

Copy link

Learn Build status updates of commit 8e909ac:

✅ Validation status: passed

File Status Preview URL Details
microsoft-edge/web-platform/tracking-prevention-images/settings-and-more-icon.png ✅Succeeded View
microsoft-edge/web-platform/tracking-prevention.md ✅Succeeded View
microsoft-edge/webview2/concepts/data-privacy-images/settings-and-more-icon.png ✅Succeeded View
microsoft-edge/webview2/concepts/data-privacy.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Copy link

Learn Build status updates of commit 400c95c:

✅ Validation status: passed

File Status Preview URL Details
microsoft-edge/web-platform/tracking-prevention-images/settings-and-more-icon.png ✅Succeeded View
microsoft-edge/web-platform/tracking-prevention.md ✅Succeeded View
microsoft-edge/webview2/concepts/data-privacy-images/settings-and-more-icon.png ✅Succeeded View
microsoft-edge/webview2/concepts/data-privacy.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cat: privacy on-hold The changes are currently on-hold and awaiting further evaluation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants