Skip to content
This repository has been archived by the owner on May 25, 2021. It is now read-only.

Privacy Policy (cont.) up2date #41

Closed
wants to merge 9 commits into from
Closed

Conversation

GrahamSH-LLK
Copy link
Member

@GrahamSH-LLK GrahamSH-LLK commented Oct 27, 2020

Continuation of #27 , resolves #24.

@WorldLanguages, you do the merge as you are the project leader. Up to date version of #27.

Copy link

@williamhorning williamhorning left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything except footer.html is fine, we just need to add a rel="noopener" attribute to the privacy policy link.

site/_includes/footer.html Outdated Show resolved Hide resolved
Copy link
Member

@Hans5958 Hans5958 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was going to do this but I'm uncertaim of the compability of this PR. Stand by.

Copy link

@williamhorning williamhorning left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing the noopener issue, this seems good to go.

@Hans5958
Copy link
Member

Closed in favour of #27.

@Hans5958 Hans5958 closed this Oct 30, 2020
@Hans5958 Hans5958 deleted the privacy-up2date branch November 15, 2020 03:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Privacy Issue] Add a disclaimer about use of Google Analytics
3 participants