-
Notifications
You must be signed in to change notification settings - Fork 30
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
Some new types #88
Some new types #88
Conversation
Not sure how to handle this one, it's basically two privacy policies that apparently both apply: <docname name="Privacy Policy">
<url name="https://policies.oath.com/us/en/oath/privacy/index.html" xpath="//main[@id='site-main']" reviewed="true">
<norecurse name="arbitrary" />
</url>
</docname>
<docname name="Flickr Privacy Policy">
<url name="https://policies.oath.com/us/en/oath/privacy/products/flickr/index.html" xpath="//main[@id='site-main']" reviewed="true">
<norecurse name="arbitrary"/>
</url>
</docname> Same for Gizmo. I'll use |
24f5c20
to
8676f97
Compare
I realized that I can't do the full tosback import until we have language support, so I'll put this PR up for review with the additions I made so far, and then we can follow up later once we have support for >=1 docs of the same type for the same service, which is what some of these need. |
I don't know all the implications of adding (too) many types that are similar to each other, but in general services use very different titles for their legal documents. Some call it Terms of Use, others call it Usage Policy. Should we allow this diversity or force a more strict small set of document types? |
This does greatly increase the number of times
is printed when you run |
Thank you very much @michielbdejong!
We encountered this issue with Skype (parent company Microsoft). In #53, we introduced a document type
Just to double-check: do you mean being able to store documents in different languages? Is the issue the fact that some documents are not in English (which contradicts our current contribution guidelines), or is it that some ToSback documents are tracked in several languages?
In order to enable comparative analysis, I would find it great to define types independently from the provider-given name. This way, we could compare terms of services or privacy policies across companies, no matter how they call them in their own lingo. This seems reinforced by the future addition of multiple languages: I don't think it would be manageable to translate each type into several languages! I understand this means being able to better define each type, so that one can find the most appropriate one beyond the names. I see a couple ways this could be solved:
This would mean, for example, that:
I am not yet sure that this characterization is a proper theoretical representation of a contractual relationship, but it does seem to help pragmatically in defining document types. What do you think of it @michielbdejong? Do you have recommendations of people to reach out to to discuss this topic? 🙂 Twitter is really the worst for that (or the best if one considers readability!), as their rules and policies are split in dozens of documents. We added a few in #85, but there are definitely many others.
I think this workaround is showing its limits. We'll probably have to find a better one. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks again @michielbdejong, this is super useful both towards integrating tosback rules but also to go further in defining types properly! 😃
fileName: 'terms_of_service' | ||
}, | ||
website_tos: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In favor of merging with tos
, unless I misunderstand and there is a difference.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's used by for instance Apple to define the ToS of the www.apple.com website, as opposed to their iChat ToS, Game Center ToS, iCloud ToS, and iTunes ToS. Maybe the right thing to do here is to define "the Apple website", iChat, Apple Game Center, iCloud and iTunes all as separate services. It's probably inaccurate to define "Apple" as a single service.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That way, we can have "the tos
of www.apple.com" instead of "the website_tos
of Apple".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe the right thing to do here is to define "the Apple website", iChat, Apple Game Center, iCloud and iTunes all as separate services.
Yes, strong +1.
This is also what I suggested and we finally decided in #77: Facebook, Facebook Ads and Facebook Payments should all be separate services. This suggests there might be a use for modeling the service provider in the end, but right now let's declare each service independently 🙂
@@ -1,12 +1,72 @@ | |||
export const TYPES = { | |||
tos: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we should expand acronyms 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep 👍 , I was thinking the same the last week.
name: 'Vendor Terms of Service', | ||
fileName: 'vendor_terms_of_service' | ||
}, | ||
tou: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In favor of merging with tos
, unless I misunderstand and there is a difference.
name: 'Acceptable Use Policy', | ||
fileName: 'acceptable_use_policy' | ||
}, | ||
termsAndConditions: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In favor of merging with tos
, unless I misunderstand and there is a difference.
name: 'Terms and Conditions', | ||
fileName: 'terms_and_conditions' | ||
}, | ||
termsAndPrivacyPolicy: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can add this, or we can ask contributors to select the subset of the document which is about ToS and about privacy, especially now that it's been made easier with #70… Could we have some examples of this @michielbdejong? 🙂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, and it applies in a few other cases as well. I'll do that.
name: 'Cookie Policy', | ||
fileName: 'cookie_policy' | ||
}, | ||
cookieAndAdPolicy: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In favor of merging with cookiesPolicy
, unless I misunderstand and there is a difference.
reviewGuidelines: { | ||
name: 'Review Guidelines', | ||
fileName: 'review_guidelines' | ||
}, | ||
cookiePolicy: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In favor of merging with cookiesPolicy
.
cookiesPolicy: { | ||
name: 'Cookies Policy', | ||
fileName: 'cookies_policy' | ||
}, | ||
copyright: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we have examples, in order to understand if this is about describing copyright owners, about DMCA, or about providing copyright licenses?
name: 'Consumer Terms of Sale', | ||
fileName: 'consumer_terms_of_sale' | ||
}, | ||
securityAndPrivacy: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In favor of merging with privacy
, unless I misunderstand and there is a difference.
Could we have an example?
I see that many of these types contain company names, and are thus trivial to realign (e.g. SAMSUNG PRIVACY POLICY FOR THE U.S. should be However some seem more complex, as they are specific to a service type, such as Host Guarantee Terms and Conditions. When these document types are not ticked, what is your intention with them? 🙂 |
Just saw that this was precisely your question in #89 😅 |
Co-authored-by: Matti Schneider <matti.schneider@diplomatie.gouv.fr>
Co-authored-by: Matti Schneider <matti.schneider@diplomatie.gouv.fr>
Will be impacted by #90, inputs welcome there! |
Superseded by #90. |
Has been only partly superseded by #90: some types are still missing. @Ndpnt and myself spent some time yesterday building a mapping table for all the document types you referenced in this PR 🙂 Since you mentioned wanting to edit the original XML files (we could also build a dynamic mapping table), this mapping table maps each ToSBack type to a CGUs type. Types in bold are new ones that should be created. In italic are documents that we believe should be dropped as the links are outdated. |
Here is a list of document types that Tosback2 uses but src/types.js does not yet list. We probably don't want to merge all of these, some include the service name in the document type and some are too similar or even look like typos. I'll tick the boxes of the ones this PR keeps, I'll update the Tosback2 rules at the XML source, not to use any document types that are not ticked here:
USAcceptable Use Policy