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

feat(google-analytics): add gatsby-plugin-google-analytics #474

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kjbrum
Copy link

@kjbrum kjbrum commented Oct 25, 2019

Still needs the actual Google Analytics tracking ID added and any other options set that are required.

This will resolve #12.

options: {
trackingId: 'GOOGLE_ANALYTICS_TRACKING_ID',
head: true,
cookieDomain: 'dsmjs.com'
Copy link
Member

Choose a reason for hiding this comment

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

since we will need to move this site from the current domain to the actual dsmjs.com domain once the site is operational, i've been hesitant to do this before we get things migrated. i wouldnt mind having confidence that its ready to go though.

since i havent configured a new site for GA for a while, do you know off the top of your head if transitioning domains is well supported or if it would be better to wait?

Copy link
Member

Choose a reason for hiding this comment

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

Will it hurt to have this config there as a placeholder, then use an actual tracking ID/set it up on Google Analytics website once this goes live on dsmjs.com?

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.

analytics
3 participants