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

Fix analyticsPixel template variable name #1087

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

alexbiehl
Copy link
Member

@alexbiehl alexbiehl commented Jun 27, 2022

This PR adresses @gbaz's comment

I merged in preperation for a redeploy, but noticed in testing that the list of pixels already created did not show up properly
in the package page. (It does show up in the page that lists all pixels for all packages for a user). So I'm going to comment > out links to this functionality for now in the release. A patch fixing that up would be welcome.

in #1042 (comment)

@fgaz
Copy link
Member

fgaz commented Jul 30, 2022

Big -1 to this and to the whole tracking pixel feature!

Google Analytics was just removed in #1097 because of privacy and legality concerns. While this is more limited from a technical pov, legally it's still the same since tracking pixels can collect personal data such as IP addresses. Worse than that, this puts control over the users' data in the hands of maintainers (who would control the tracking settings), while putting the responsibility in the hands of hackage operators and haskell.org (data controller)! That's a huge burden and has too much potential to be abused!

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