-
-
Notifications
You must be signed in to change notification settings - Fork 33
Added Cloudflare for Review #215
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
Conversation
doamatto
left a comment
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.
Sorry for the late review— it's a good start, but there's more work that needs to be done
products/cloudflare.toml
Outdated
| "We do not allow our third-party service providers to use your personal data for their own purposes or to sell your personal data. We only permit them to process your personal data for specified purposes and in accordance with our instructions.” Source: Cloudflare Candidate Privacy Notice", | ||
| "Cloudflare shall not subcontract any of its processing operations performed on behalf of the Customer under the Clauses without the prior written consent of the Customer.” Source: Cloudflare Data Processing Addendum" | ||
| ] | ||
| notes = [ |
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.
There should be/is a more graceful way to handle attributing quotes
Thanks for the review. Can you take a look again. I've made certain changes. |
Type of pull request: product addition
Related issues: Issue #207 (Add Cloudflare)
Please review this pull request and let me know if any changes are needed.