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

Privacy: Expand Compose, revise Copilot #3206

Merged
merged 15 commits into from
Jul 9, 2024

Conversation

majenny
Copy link
Contributor

@majenny majenny commented Jun 24, 2024

microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
@mikehoffms mikehoffms changed the title Adding privacy entry for Compose inline rewrite Privacy: Expand Compose, revise Copilot Jul 1, 2024
Copy link
Contributor

@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.

Approved PR per Writer/Editor review, after revisions were made.

@majenny
Copy link
Contributor Author

majenny commented Jul 1, 2024

This PR LGTM, thanks all for reviewing!

Copy link
Contributor

@sanketj sanketj left a comment

Choose a reason for hiding this comment

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

LGTM with one suggestion.

microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
microsoft-edge/privacy-whitepaper/index.md Outdated Show resolved Hide resolved
@mikehoffms mikehoffms requested a review from apryzgoda July 2, 2024 19:14
Copy link

Learn Build status updates of commit 25f8f70:

✅ Validation status: passed

File Status Preview URL Details
microsoft-edge/privacy-whitepaper/index-images/compose.png ✅Succeeded n/a (file deleted or renamed)
microsoft-edge/privacy-whitepaper/index-images/settings-and-more-icon.png ✅Succeeded View
microsoft-edge/privacy-whitepaper/index-images/summarize-content.png ✅Succeeded n/a (file deleted or renamed)
microsoft-edge/privacy-whitepaper/index.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

@captainbrosset captainbrosset merged commit abd9eb3 into main Jul 9, 2024
2 checks passed
@captainbrosset captainbrosset deleted the user/majenny/ComposeInlinePrivacy branch July 9, 2024 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants