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

Add Fathom Analytics to v5 docs #33904

Merged
merged 2 commits into from
May 13, 2021
Merged

Add Fathom Analytics to v5 docs #33904

merged 2 commits into from
May 13, 2021

Conversation

mdo
Copy link
Member

@mdo mdo commented May 8, 2021

No description provided.

@mdo mdo added this to Inbox in v5.0.1 via automation May 8, 2021
@XhmikosR XhmikosR added docs and removed meta labels May 11, 2021
@@ -5,3 +5,4 @@
ga('send', 'pageview');
</script>
<script async src="https://www.google-analytics.com/analytics.js"></script>
<script defer src="https://cdn.usefathom.com/script.js" data-site="TAXRUMOF"></script>
Copy link
Member

Choose a reason for hiding this comment

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

Any specific reason this is defer and not async?

Copy link
Member Author

Choose a reason for hiding this comment

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

It's what they included in their code snippet. We can try to switch it around.

Copy link
Member

Choose a reason for hiding this comment

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

@mdo we should drop GA, shouldn't we?

@mdo
Copy link
Member Author

mdo commented May 13, 2021

Shipping as-is for now—will revisit for v5.1.0.

@mdo mdo merged commit 3464d3c into main May 13, 2021
v5.0.1 automation moved this from Inbox to Done May 13, 2021
@mdo mdo deleted the v5-fathom branch May 13, 2021 04:34
@XhmikosR XhmikosR added this to Inbox in v4.6.1 via automation May 13, 2021
@XhmikosR XhmikosR moved this from Inbox to Cherry-picked/Manually backported in v4.6.1 May 14, 2021
XhmikosR pushed a commit that referenced this pull request May 14, 2021
* Add Fathom Analytics to v5 docs

* newline
@XhmikosR XhmikosR removed this from Cherry-picked/Manually backported in v4.6.1 May 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v5.0.1
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants