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

Update X (Formerly Known As Twitter) #180

Merged
merged 9 commits into from
Oct 2, 2024

Conversation

JTModel3
Copy link
Contributor

Type of pull request: Product Edit

Related issues: Outdated, especially after the company was bought by Elon Musk, and was turned into a private Company.


Update every section to accurately represent what the current X / Twitter's privacy policy is.

@milesmcc milesmcc requested a review from doamatto November 22, 2023 16:32
Copy link
Collaborator

@doamatto doamatto left a comment

Choose a reason for hiding this comment

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

Here's what I've got change wise. GitHub's website is struggling for me, so this I can't get the comments on the specific lines but here you go :

  • There's no need for "\" on behavioral-marketing, third-party-collection, data-deletion, third-party-access, law-enforcement, revision-notify
  • rubric.history— last modified is as top of page, previous policies at bottom
  • rubric.data-deletion— add note with link to wiki page
  • rubric.third-party-access— there are some subprocessors listed here, maybe link to it in a note ?
  • rubric.noncritical-purposes— see §5.3
  • rubric.list-collected— seeing a fair amount of "such as" terminology in §1.2, shouldn't be yes in that case

Once that gets fixed, I'll give it another once-over sometime next week and get it merged. Fix any issues CI prompts, too.

@doamatto doamatto self-assigned this Oct 2, 2024
@doamatto doamatto added product Related to a product on PrivacySpy size/XS labels Oct 2, 2024
@doamatto doamatto dismissed their stale review October 2, 2024 17:41

Will finish PR myself

@doamatto doamatto merged commit c3aa110 into Politiwatch:master Oct 2, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product Related to a product on PrivacySpy size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants