Skip to content

Conversation

GDefrevillePEReN
Copy link
Contributor

No description provided.

Copy link
Member

@Cli4d Cli4d left a comment

Choose a reason for hiding this comment

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

Hi @GDefrevillePEReN

Thank you for making this contribution. I have reviewed it and made some suggested changes to make on your fork. Once that is done I'll proceed to merge the PR

Comment on lines +1 to +12
{
"name": "Tesla",
"documents": {
"Privacy Policy": {
"fetch": "https://www.tesla.com/fr_fr/legal/privacy",
"select": {
"startBefore": ".tds-shell h1",
"endBefore": ".tds-shell footer"
}
}
}
}
Copy link
Member

Choose a reason for hiding this comment

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

Hi @GDefrevillePEReN I noticed these selectors where including the header. So I found a simpler selector to use. Also fetching was bringing up a 403, so I enabled client scripts to resolve that.

Given this is your fork, I cannot actualize these changes. So you can add this into the file to bring on board the changes I talked about

{
  "name": "Tesla",
  "documents": {
    "Privacy Policy": {
      "fetch": "https://www.tesla.com/fr_fr/legal/privacy",
      "select": [ 
        "#tdsonecolumn--UJnUWI7OCdU"
      ],
      "executeClientScripts": true
    }
  }
}

@Cli4d
Copy link
Member

Cli4d commented Aug 12, 2025

Back to review this PR after the earlier suggested changes. It seems the website blocked our bot as it brings up a 403 error. So I will leave this open for a while and if tracking can't resume I will proceed to close this PR as unmerged

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.

2 participants