-
Notifications
You must be signed in to change notification settings - Fork 985
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 Browser Analytics #779
Comments
Surely we want this to be active on launch? |
Yea we want this active on launch, but it's trivial to do so it shouldn't be hard, it'll just be dropping in a bit of JS into the template. The only hard part is our CSP policy. |
I asked for some clarification in IRC in We already have metrics on legacy PyPI, so it's important to have analytics active in Warehouse at launch, so we don't miss data during the switch. To abide by the content security policy, we need to figure out how to do GA without chunking some inline JS; our CSP policy instructs browsers to reject inline JavaScript in the HTML, only [allowing] linked scripts. In the worst case, we could relax the restrictions, but Donald would rather not do that. |
We'll want to add Google Analytics or something so that we can see traffic and browsers and such. This will most likely largely be done in
pypi-theme
.The text was updated successfully, but these errors were encountered: