Skip to content

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

Merged
hugovk merged 1 commit into
python:mainfrom
ewdurbin:analytics
Apr 17, 2025
Merged

Docs: Migrate entirely to PSF hosted plausible for analytics#132648
hugovk merged 1 commit into
python:mainfrom
ewdurbin:analytics

Conversation

@ewdurbin

@ewdurbin ewdurbin commented Apr 17, 2025

Copy link
Copy Markdown
Member

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

hugovk commented Apr 17, 2025

Copy link
Copy Markdown
Member

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

@ewdurbin

Copy link
Copy Markdown
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
@miss-islington-app

Copy link
Copy Markdown

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
…H-132648)

(cherry picked from commit 1d529cb)

Co-authored-by: Ee Durbin <ewdurbin@gmail.com>
@bedevere-app

bedevere-app Bot commented Apr 17, 2025

Copy link
Copy Markdown

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

hugovk commented Apr 17, 2025

Copy link
Copy Markdown
Member

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

Yhg1s commented Apr 17, 2025

Copy link
Copy Markdown
Member

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
Copy Markdown

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
…H-132648)

(cherry picked from commit 1d529cb)

Co-authored-by: Ee Durbin <ewdurbin@gmail.com>
@bedevere-app

bedevere-app Bot commented Apr 17, 2025

Copy link
Copy Markdown

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 <ewdurbin@gmail.com>
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 <ewdurbin@gmail.com>
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