Skip to content

Docs: Migrate entirely to PSF hosted plausible for analytics #132648

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

Merged
merged 1 commit into from
Apr 17, 2025

Conversation

ewdurbin
Copy link
Member

@ewdurbin ewdurbin commented Apr 17, 2025

I also updated the script to enable tracking of outbound link clicks, which may be nice to have.

Analytics can still be viewed publicly at https://analytics.python.org/docs.python.org.

The only discrepency I have encountered comparing the analytics data between the two is that hosted plausible seems to count more Unique visitors. I'm unsure why... but for our purposes it isn't really important, as we are mainly tracking what pages/languages are being viewed. edit: this is now fixed, just needed to map Fastly-Client-IP header to the X-Plausible-IP header. I'll reset stats and re-import to clear things up.


📚 Documentation preview 📚: https://cpython-previews--132648.org.readthedocs.build/

I also updated the script to enable tracking of outbound link clicks, which may be nice to have.

Analytics can still be viewed publicly at https://analytics.python.org/docs.python.org.

The only discrepency I have encountered comparing the analytics data between the two is that hosted plausible seems to count more Unique visitors. I'm unsure why... but for our purposes it isn't really important, as we are mainly tracking what pages/languages are being viewed.
@hugovk
Copy link
Member

hugovk commented Apr 17, 2025

We'll also need to update https://github.com/pypa/packaging.python.org

@ewdurbin
Copy link
Member Author

We'll also need to update https://github.com/pypa/packaging.python.org

I oped this first :) pypa/packaging.python.org#1850

@hugovk hugovk added the needs backport to 3.13 bugs and security fixes label Apr 17, 2025
@hugovk hugovk merged commit 1d529cb into python:main Apr 17, 2025
37 checks passed
@miss-islington-app
Copy link

Thanks @ewdurbin for the PR, and @hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Apr 17, 2025
@bedevere-app
Copy link

bedevere-app bot commented Apr 17, 2025

GH-132660 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Apr 17, 2025
@hugovk
Copy link
Member

hugovk commented Apr 17, 2025

We still have Plausible-hosted on the 3.12 branch but it's in security only.

This is more an infra change than a bug or security fix, so I think it's worth doing.

@Yhg1s is that okay?

@Yhg1s
Copy link
Member

Yhg1s commented Apr 17, 2025

Yeah, it makes sense to backport this to 3.12.

@Yhg1s Yhg1s added the needs backport to 3.12 only security fixes label Apr 17, 2025
@miss-islington-app
Copy link

Thanks @ewdurbin for the PR, and @hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Apr 17, 2025
@bedevere-app
Copy link

bedevere-app bot commented Apr 17, 2025

GH-132664 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 only security fixes label Apr 17, 2025
AA-Turner pushed a commit that referenced this pull request Apr 18, 2025
…H-132648) (#132660)

Docs: Migrate entirely to PSF hosted plausible for analytics (GH-132648)
(cherry picked from commit 1d529cb)

Co-authored-by: Ee Durbin <[email protected]>
hugovk pushed a commit that referenced this pull request Apr 18, 2025
…H-132648) (#132664)

Docs: Migrate entirely to PSF hosted plausible for analytics (GH-132648)
(cherry picked from commit 1d529cb)

Co-authored-by: Ee Durbin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants