FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Document that we have Docs Analytics using Plausible by StanFromIreland · Pull Request #1496 · python/devguide · GitHub

Document that we have Docs Analytics using Plausible - #1496

Open
StanFromIreland wants to merge 4 commits into
python:mainfrom
StanFromIreland:patch-analytics
Open

Document that we have Docs Analytics using Plausible#1496
StanFromIreland wants to merge 4 commits into
python:mainfrom
StanFromIreland:patch-analytics

Conversation

StanFromIreland commented Jan 18, 2025
edited by hugovk
Loading

Copy link
Copy Markdown
Member

#1479

Created a new file for this as I was unable to find a good place for it in the existing documentation as it is most of tutorial form.

I think a separate file is good for expanding on this in the future, maybe more detail about the cookie and privacy concerns etc.?


📚 Documentation preview 📚: https://cpython-devguide--1496.org.readthedocs.build/

hugovk commented Jan 18, 2025

Copy link
Copy Markdown
Member

Thanks for the PR!

Hmm, I'm not sure if this needs a new page, for a single sentence? If we did expand it about cookies/privacy, I think that would mostly be to link to Plausible's own detailed descriptions on their own website.

@nedbat: related to the restructuring work, where do you think this would fit best? We might as well put it there already.


By the way, we don't need to prefix PRs with "gh-xxxx" in this repo.

AA-Turner changed the title gh-1479: Document that we have Docs Analytics using Plausible Document that we have Docs Analytics using Plausible Jan 19, 2025
Comment thread documentation/index.rst Outdated

nedbat commented Jan 20, 2025

Copy link
Copy Markdown
Member

@hugovk Thanks for the heads-up, but this is a small enough change to accommodate later no matter what.

Copy link
Copy Markdown
Member Author

@nedbat friendly ping :-)

Copy link
Copy Markdown
Member

Hugo's feedback is still outstanding:

Hmm, I'm not sure if this needs a new page, for a single sentence? If we did expand it about cookies/privacy, I think that would mostly be to link to Plausible's own detailed descriptions on their own website.

Please merge into a pre existing page.

Copy link
Copy Markdown
Member Author

@AA-Turner Ned didn't seem to mind. What page do you propose :-)

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

5 participants


Back | FazBrowse Home | New Git URL