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

Enh: Remove Google Tag #124

Merged
merged 1 commit into from
Dec 6, 2023
Merged

Conversation

sneakers-the-rat
Copy link
Contributor

Currently the site embeds google analytics (twice actually, once for the gtagjs extension and another for the pydata theme).

Elsewhere, we give a rather stringent standard for opt-in data collection: https://www.pyopensci.org/software-peer-review/about/package-scope.html#telemetry-user-informed-consent

I think we could stand to do the same on our websites, no?

There is another tracker, matomo, that still attempts to embed itself in the page: https://github.com/pyOpenSci/python-package-guide/blob/main/_static/matomo.js
so analytics needs managed, hopefully?

Making this PR as a proposal, let's lead the way out of the darkness of surveillance web <3

@lwasser
Copy link
Member

lwasser commented Dec 6, 2023

@sneakers-the-rat thank you! i moved all of our analytics to matomo so this is great to remove GA everywhere. it's what ropensci uses as well.

we probably need to make the same move in our peer review guide as well if you're up for a second PR.

what i could also use help with (IF you know how to make this happen) is ropensci has the "cookies notice" when you go to the site allowing the user to chose. do you happen to know how to set that up? i know it works with matomo. and i know matomo can be compliant (but correct me please if i'm wrong!!)

@lwasser lwasser self-requested a review December 6, 2023 15:57
Copy link
Member

@lwasser lwasser left a comment

Choose a reason for hiding this comment

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

Approving this and merging - thank you @sneakers-the-rat for this. i totally missed the last step of removing those deps and the GA code!

@lwasser lwasser changed the title [Proposal] Remove Google Tag Enh: Remove Google Tag Dec 6, 2023
@lwasser lwasser merged commit 889b582 into pyOpenSci:main Dec 6, 2023
3 checks passed
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.

2 participants