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

Update Google Analytics ID #3700

Merged
merged 4 commits into from
Apr 21, 2021
Merged

Update Google Analytics ID #3700

merged 4 commits into from
Apr 21, 2021

Conversation

NicolasHug
Copy link
Member

This PR updates the GA id to UA-117752657-2 as recommended internally.
Also now that GA is natively supported by the theme (pytorch/pytorch_sphinx_theme#110) we don't need to tweak the template anymore.

CC @brianjo

@NicolasHug NicolasHug marked this pull request as draft April 20, 2021 19:24
@NicolasHug
Copy link
Member Author

The source of the rendered docs is properly updated with the new id:, see e.g. https://527155-73328905-gh.circle-artifacts.com/0/docs/genindex.html

@NicolasHug NicolasHug marked this pull request as ready for review April 20, 2021 23:19
Copy link
Member

@fmassa fmassa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for looking into it!

@fmassa fmassa merged commit 94e1919 into pytorch:master Apr 21, 2021
facebook-github-bot pushed a commit that referenced this pull request May 4, 2021
Summary:
* Phew

* not sure CI got properly triggered?

Reviewed By: NicolasHug

Differential Revision: D28169156

fbshipit-source-id: 2bafba64c934fb19a76b29d5a058decb7f03e9e6
facebook-github-bot pushed a commit to pytorch/pytorch that referenced this pull request May 4, 2021
Summary:
This PR fixes the GA id and relies on `pytorch-sphinx-theme`  to set the GA script instead of hard-coding it (this is supported since pytorch/pytorch_sphinx_theme#110 was merged).

Similar PRs were opened and merged in torchchvision/audio/text, e.g.: pytorch/vision#3700

CC brianjo

Pull Request resolved: #56578

Reviewed By: mrshenli

Differential Revision: D28199244

Pulled By: ranman

fbshipit-source-id: a20b7fd1b1da3ebff491286c3eeb1410f3c80670
krshrimali pushed a commit to krshrimali/pytorch that referenced this pull request May 19, 2021
Summary:
This PR fixes the GA id and relies on `pytorch-sphinx-theme`  to set the GA script instead of hard-coding it (this is supported since pytorch/pytorch_sphinx_theme#110 was merged).

Similar PRs were opened and merged in torchchvision/audio/text, e.g.: pytorch/vision#3700

CC brianjo

Pull Request resolved: pytorch#56578

Reviewed By: mrshenli

Differential Revision: D28199244

Pulled By: ranman

fbshipit-source-id: a20b7fd1b1da3ebff491286c3eeb1410f3c80670
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