Skip to content

crds-analytics-wrapper integration#3492

Open
jacobtellep wants to merge 7 commits intodemofrom
feature/implement-analytics-wrapper
Open

crds-analytics-wrapper integration#3492
jacobtellep wants to merge 7 commits intodemofrom
feature/implement-analytics-wrapper

Conversation

@jacobtellep
Copy link
Copy Markdown
Contributor

@jacobtellep jacobtellep commented Aug 26, 2025 •
edited
Loading

Problem

Move to using crds-analytics-wrapper

Solution

Replaced Segment integration with Crossroads Analytics wrapper and updated includes:

  • Modified _includes/_head.html and _includes/_footer-script.html to load the wrapper.
  • Updated .env-sample with required analytics env vars.
  • Diff: 3 files changed, 74 insertions, 12 deletions.
  • Files: .env-sample, _includes/_head.html, _includes/_footer-script.html.

Corresponding Branch

N/A

Testing

  • Install deps and serve: bundle install && npm i && bundle exec jekyll serve.
  • Load any page; view source/Network:
    • Confirm wrapper loads (no requests to Segment domains).
    • Verify analytics object is present and initialized.
  • Optional: run npm run build to ensure assets build cleanly.

@netlify
Copy link
Copy Markdown

netlify Bot commented Aug 26, 2025 •
edited
Loading

✅ Deploy Preview for demo-crds-jekyll ready!

Name Link
🔨 Latest commit 59651bf
🔍 Latest deploy log https://app.netlify.com/projects/demo-crds-jekyll/deploys/68af41566ca48f0007aa9fe4
😎 Deploy Preview https://deploy-preview-3492--demo-crds-jekyll.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

jacobtellep and others added 6 commits August 27, 2025 11:20
@jacobtellep @claude
Remove crossorigin="anonymous" from analytics wrapper script
c64321f 
Removes the crossOrigin attribute that was causing CORS issues with the analytics wrapper script loading.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@jacobtellep
Refactor Crossroads Analytics wrapper to improve initialization and e…
b1ba045 
…rror handling
@jacobtellep @claude
Clean up analytics implementation and fix critical issues
973ce35 
- Remove all debugging code and console logging
- Add browser polyfills for Node.js globals (process, global)
- Fix script loading race condition with proper event handlers
- Add robust error handling and method name fallbacks
- Improve page tracking logic for navigation scenarios
- Remove crossorigin attribute (already committed separately)

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@jacobtellep
Enhance Crossroads Analytics integration with improved initialization…
dd770a0 
… and auto-tracking setup; add data attributes for article engagement tracking
@jacobtellep
create env var for write key
a3f4ef9
@jacobtellep
add temp logging
59651bf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jodylane jodylane jodylane approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jacobtellep @jodylane