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 Comptoir des Cotonniers Privacy Policy #529

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

Conversation

OTA-Bot
Copy link
Contributor

@OTA-Bot OTA-Bot commented Sep 29, 2022

🔎 Inspect this declaration update suggestion


Bots should take care of checking the formatting and the validity of the declaration. As a human reviewer, you should check:

  • Selectors are:
    • stable: as much as possible, the CSS selectors are meaningful and specific (e.g. .tos-content rather than .ab23 .cK_drop > div).
    • simple: the CSS selectors do not have unnecessary specificity (e.g. if there is an ID, do not add a class or a tag).
  • Generated version is:
    • relevant: it is not just a series of links, for example.
    • readable: it is complete and not mangled.
    • clean: it does not contain navigation links, unnecessary images, or extra content.

Thanks to your work and attention, Open Terms Archive will ensure that high quality data is available for all reusers, enabling them to do their part in shifting the balance of power towards end users and regulators instead of spending time collecting and cleaning documents 💪


This update suggestion has been created through the Contribution Tool, which enables graphical declaration of documents.
You can load it on your local instance if you have one set up.

@OTA-Bot
Copy link
Contributor Author

OTA-Bot commented Sep 29, 2022

🔎 Inspect the updated declaration suggestion


A new suggestion has been made to update this declaration, voiding the previous ones.

As a human reviewer, here are the things you should check:

  • Selectors are:
    • stable: as much as possible, the CSS selectors are meaningful and specific (e.g. .tos-content rather than .ab23 .cK_drop > div).
    • simple: the CSS selectors do not have unnecessary specificity (e.g. if there is an ID, do not add a class or a tag).
  • Generated version is:
    • relevant: it is not just a series of links, for example.
    • readable: it is complete and not mangled.
    • clean: it does not contain navigation links, unnecessary images, or extra content.

This suggestion has been created through the Contribution Tool, which enables graphical declaration of documents.
You can load it on your local instance if you have one set up.

@martinratinaud
Copy link
Member

@clementbiron I changed the URL for this as it was not available anymore BUT this URL contains old in it so I'm not sure it's the right move.

But the other I've found whil looking on Google was from talent.comptoirdescottoniers.com, so I would love your idea on that

@OTA-Bot
Copy link
Contributor Author

OTA-Bot commented Sep 29, 2022

🔎 Inspect the updated declaration suggestion


A new suggestion has been made to update this declaration, voiding the previous ones.

As a human reviewer, here are the things you should check:

  • Selectors are:
    • stable: as much as possible, the CSS selectors are meaningful and specific (e.g. .tos-content rather than .ab23 .cK_drop > div).
    • simple: the CSS selectors do not have unnecessary specificity (e.g. if there is an ID, do not add a class or a tag).
  • Generated version is:
    • relevant: it is not just a series of links, for example.
    • readable: it is complete and not mangled.
    • clean: it does not contain navigation links, unnecessary images, or extra content.

This suggestion has been created through the Contribution Tool, which enables graphical declaration of documents.
You can load it on your local instance if you have one set up.

@martinratinaud
Copy link
Member

Fixes #194

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