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

docs: enable plausible.js analytics #1822

Merged
merged 1 commit into from
Oct 24, 2022
Merged

Conversation

agoose77
Copy link
Collaborator

@agoose77 agoose77 commented Oct 22, 2022

Fixes #1819

This was particularly straightforward because the PyData Sphinx Theme already supports Plausible.js, and our redirect system uses pre-rendered HTML (and therefore is not injected with the analytics JS).


📚 The documentation for this PR will be available at https://awkward-array.readthedocs.io/en/agoose77-docs-add-analytics/ once Read the Docs has finished building 🔨

@agoose77 agoose77 requested a review from jpivarski October 22, 2022 13:37
@agoose77 agoose77 changed the title feat: enable plausible.js analytics docs: enable plausible.js analytics Oct 22, 2022
Copy link
Member

@jpivarski jpivarski left a comment

Choose a reason for hiding this comment

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

Even better! I checked the output and found this in the <head>:

  <!-- Pre-loaded scripts that we'll load fully later -->
  <link rel="preload" as="script" href="_static/scripts/pydata-sphinx-theme.js?digest=1e1de1a1873e13ef5536">

    <script data-url_root="./" id="documentation_options" src="_static/documentation_options.js"></script>
    <script src="_static/jquery.js"></script>
    <script src="_static/underscore.js"></script>
    <script src="_static/doctools.js"></script>
    <script src="_static/clipboard.min.js"></script>
    <script src="_static/copybutton.js"></script>
    <script src="_static/jupyterlite_sphinx.js"></script>
    <script src="_static/design-tabs.js"></script>
    <script data-domain="awkward-array.org" defer="defer" src="https://views.scientific-python.org/js/plausible.js"></script>
    <script async="async" src="/_/static/javascript/readthedocs-doc-embed.js"></script>
    <script>DOCUMENTATION_OPTIONS.pagename = 'index';</script>
    <link rel="canonical" href="https://awkward-array.readthedocs.io/en/stable/index.html" />
    <link rel="index" title="Index" href="genindex.html" />
    <link rel="search" title="Search" href="search.html" />
    <link rel="next" title="Getting started" href="getting-started/index.html" />
  <meta name="viewport" content="width=device-width, initial-scale=1" />
  <meta name="docsearch:language" content="en">
  
<!-- RTD Extra Head -->

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.

Add statistics tracker to the new website
2 participants