Skip to content

Fix Microsoft Copilot#338

Open
PierreMesure wants to merge 1 commit into
OpenTermsArchive:mainfrom
PierreMesure:fix-copilot
Open

Fix Microsoft Copilot#338
PierreMesure wants to merge 1 commit into
OpenTermsArchive:mainfrom
PierreMesure:fix-copilot

Conversation

@PierreMesure

Copy link
Copy Markdown
Contributor

Fixes #137 and another silent issue with the privacy policy.

The links in the declaration were redirecting so I changed to the final URLs and updated the selectors for very broad ones. We can discuss whether they are too broad before merging, the side menu is included in the privacy policy.

@MattiSG MattiSG left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Exciting to see all these contributions!

"remove": [
".cdx_mkt_brand"
],
"fetch": "https://www.microsoft.com/en-us/microsoft-copilot/for-individuals/termsofuse",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Beware, you're switching from en-ie to en-us, while the declared jurisdiction of this collection is EU! This URL is more likely to yield the US version of the terms.

"#dvServicesAgreement"
]
"fetch": "https://www.microsoft.com/en-us/privacy/privacystatement",
"select": "main"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Indeed, the navigation menu is included. Maybe try with a remove of a range selector with startBefore: ".inPageNavigationV2" and endAfter: ".col"? 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Microsoft CopilotTerms of Service ‧ not tracked anymore

2 participants