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 npm Public Registry Privacy Policy #23

Merged
merged 5 commits into from
Dec 19, 2023

Conversation

OTA-Bot
Copy link

@OTA-Bot OTA-Bot commented Nov 22, 2023

🔎 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.
  • validUntil date is correctly input:
    • Check the latest versions
    • Find the first occurence of the problematic change
    • Note the creation date of this issue (inspect x days ago with developer tools and copy datetime)
    • Find the latest snapshot from the list of snapshots strictly anterior to this date
    • Copy the creation date of this snapshot (inspect x days ago with developer tools and copy datetime)
    • Replace the to-be-determined date in history file directly from GitHub or check out branch update_npm_public_registry_privacy_policy and do so on your local

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.

@clementbiron clementbiron requested a review from MattiSG November 22, 2023 13:19
@MattiSG
Copy link
Member

MattiSG commented Nov 22, 2023

Please make sure that the content is complete before asking for a review 😉 the date is “to be determined”

declarations/npm Public Registry.history.json Outdated Show resolved Hide resolved
declarations/npm Public Registry.json Outdated Show resolved Hide resolved
@clementbiron clementbiron force-pushed the update_npm_public_registry_privacy_policy branch from 614e7b5 to 0dc02c9 Compare December 19, 2023 10:09
@clementbiron clementbiron merged commit 3772a1d into main Dec 19, 2023
3 checks passed
@clementbiron clementbiron deleted the update_npm_public_registry_privacy_policy branch December 19, 2023 10:10
@clementbiron clementbiron linked an issue Dec 19, 2023 that may be closed by this pull request
1 task
@MattiSG MattiSG mentioned this pull request Dec 19, 2023
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.

npm Public RegistryPrivacy Policy ‧ not tracked anymore
3 participants