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

Add privacy page #25

Closed
wants to merge 1 commit into from

Conversation

ivanayov
Copy link

With this change Google Analytics can be enabled. Previously this
was restricted due to GDPR

Needs little tuning before merge.

Signed-off-by: Ivana Yovcheva [email protected]

Resolves #24

With this change  Google Analytics can be enabled. Previously this
was restricted due to GDPR

Signed-off-by: Ivana Yovcheva <[email protected]>
@alexellis
Copy link
Member

Hi, where did the text come from for this?

Privacy policy is probably best placed in footer then as a main nav item too.

Alex

@ivanayov
Copy link
Author

@alexellis I took the template text from Wordpress docs and edited it where it wasn't appropriate

@LucasRoesler
Copy link
Member

Are the "Ads" and "Comments" sections needed, since we don't have either on the site?

@alexellis
Copy link
Member

I don't think that Wordpress is a suitable template. The blog isn't a Wordpress site, it's a static HTML page with Google Analytics. That's the only thing a guide has to cover. Also "OpenFaaS" is not an entity.

@alexellis alexellis closed this Aug 28, 2018
@alexellis
Copy link
Member

Derek add label: invalid

@derek derek bot added the invalid label Aug 28, 2018
@alexellis
Copy link
Member

I'm sorry to close this, I know it had good intentions but I am worried that someone will find this and assume that the wording/text applies. A privacy policy has to be properly researched and not plagiarized (read copy/pasted) from a Wordpress blog.

I will update the issue to make it clear this is one for me to pick up.

@ivanayov ivanayov deleted the ivanayov/privacy_policy branch August 28, 2018 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants