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

chore: added script to dev site home #1875

Merged
merged 3 commits into from
Nov 9, 2021
Merged

chore: added script to dev site home #1875

merged 3 commits into from
Nov 9, 2021

Conversation

jpvajda
Copy link
Contributor

@jpvajda jpvajda commented Nov 8, 2021

This PR attempts to add the crazy egg script to the developer site home page as well, so both the / page an /instant-observability/ both have the script running.

@jpvajda jpvajda linked an issue Nov 8, 2021 that may be closed by this pull request
4 tasks
@gatsby-cloud
Copy link

gatsby-cloud bot commented Nov 8, 2021

Gatsby Cloud Build Report

developer-website-develop

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 2m

Performance

Lighthouse report

Metric Score
Performance 🔶 27
Accessibility 💚 90
Best Practices 🔶 87
SEO 🔶 76

🔗 View full report

@jpvajda
Copy link
Contributor Author

jpvajda commented Nov 9, 2021

This approach to use / as the path with an || seems to throw the same errors as using a ternary operator...

Uncaught Error: CE: multiple userscripts installed
    <anonymous> http://script.crazyegg.com/pages/scripts/0045/9836.js:1
9836.js:1:70
    <anonymous> http://script.crazyegg.com/pages/scripts/0045/9836.js:1

@jpvajda
Copy link
Contributor Author

jpvajda commented Nov 9, 2021

I've added a path for index.html, but it seems the script is still not firing on the index page in local testing

@jpvajda jpvajda merged commit e484024 into develop Nov 9, 2021
@jpvajda jpvajda deleted the crazy-egg branch November 9, 2021 22:59
@nr-opensource-bot
Copy link
Contributor

🎉 This PR is included in version 1.69.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Dev Site] Add crazyegg tracking to script to Devsite home
4 participants