Skip to content

Fix CTA and snippet copy event tracking#363

Merged
shilman merged 5 commits into
mainfrom
sidnioulz/growth-add-missing-cta-plausible-tracker
Dec 8, 2025
Merged

Fix CTA and snippet copy event tracking#363
shilman merged 5 commits into
mainfrom
sidnioulz/growth-add-missing-cta-plausible-tracker

Conversation

@Sidnioulz
Copy link
Copy Markdown
Contributor

@Sidnioulz Sidnioulz commented Dec 3, 2025

  • Add missing CTA tracker for sidebar
  • Add missing code copy tracker for frontpage hero
  • Fix bug in prop tracking for CodeSnippets (added snippetPath)
  • Add custom event on top of CodeSnippetCopy for CodeSnippets to help with funnel building

@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 3, 2025

Deploy Preview for storybook-addon-catalog ready!

Name Link
🔨 Latest commit 877fd3c
🔍 Latest deploy log https://app.netlify.com/projects/storybook-addon-catalog/deploys/69367c4bb14e580008fcca39
😎 Deploy Preview https://deploy-preview-363--storybook-addon-catalog.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.

@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 3, 2025

Deploy Preview for storybook-frontpage ready!

Name Link
🔨 Latest commit 877fd3c
🔍 Latest deploy log https://app.netlify.com/projects/storybook-frontpage/deploys/69367c4b32fd5c000881eacc
😎 Deploy Preview https://deploy-preview-363--storybook-frontpage.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.

@Sidnioulz Sidnioulz force-pushed the sidnioulz/growth-add-missing-cta-plausible-tracker branch 4 times, most recently from 8e5f2fe to cefa2fc Compare December 3, 2025 15:39
@Sidnioulz Sidnioulz requested a review from shilman December 3, 2025 16:05
@Sidnioulz Sidnioulz changed the title fix: Add missing CTA tracker for sidebar Fix issues with event tracking for growth experiment Dec 3, 2025
@Sidnioulz
Copy link
Copy Markdown
Contributor Author

@shilman I pushed a few more fixes. Builds are broken due to a recent Netlify change that I've notified @kylegach about. Feel free to merge while I'm away if this looks good to you!

Copy link
Copy Markdown
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

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

🙌 thank you!

@Sidnioulz Sidnioulz force-pushed the sidnioulz/growth-add-missing-cta-plausible-tracker branch from b8ff3b7 to e7c6b6a Compare December 5, 2025 11:22
@Sidnioulz
Copy link
Copy Markdown
Contributor Author

@kylegach could you please confirm if it's okay to merge this? Do I need to manually deploy stuff on Netlify after merging?

Comment thread apps/frontpage/app/event-log/route.ts Outdated
Comment thread apps/frontpage/app/event-log/route.ts Outdated
@shilman shilman changed the title Fix issues with event tracking for growth experiment Fix CTA and snippet copy event tracking Dec 8, 2025
@shilman shilman merged commit 171a169 into main Dec 8, 2025
12 checks passed
@shilman shilman deleted the sidnioulz/growth-add-missing-cta-plausible-tracker branch December 8, 2025 08:08
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.

2 participants