Skip to content

refactor: update button click tracking events (backport of #2104)#2110

Merged
edwinjosechittilappilly merged 3 commits into
mainfrom
backport/2104-button-click-tracking
Jul 17, 2026
Merged

refactor: update button click tracking events (backport of #2104)#2110
edwinjosechittilappilly merged 3 commits into
mainfrom
backport/2104-button-click-tracking

Conversation

@edwinjosechittilappilly

@edwinjosechittilappilly edwinjosechittilappilly commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

Backports #2104 from release-saas-ga-0.6.2 to main.

Test plan

  • CI passes

Summary by CodeRabbit

  • Analytics
    • Updated button click tracking to use the “CTA Clicked” event name.
    • CTA details are now required when recording button interactions.

@github-actions github-actions Bot added frontend 🟨 Issues related to the UI/UX refactor labels Jul 16, 2026
@coderabbitai

coderabbitai Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: a2a7b1ff-eca7-42b8-b51a-e19c8021e111

📥 Commits

Reviewing files that changed from the base of the PR and between 57b1fe7 and e913e09.

📒 Files selected for processing (1)
  • frontend/lib/analytics.ts

Walkthrough

trackButton now requires a CTA value and emits "CTA Clicked" instead of "Button Clicked", while retaining existing action defaults and property merging.

Changes

CTA analytics

Layer / File(s) Summary
Analytics contract and event emission
frontend/lib/analytics.ts
ButtonEventParams.CTA is required, and trackButton emits "CTA Clicked" with the existing payload behavior.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested labels: bug

Suggested reviewers: mfortman11

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: updating button click tracking events.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch backport/2104-button-click-tracking

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mpawlow mpawlow left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Code Review 1

  • ✅ Approved / LGTM 🚀

@github-actions github-actions Bot added the lgtm label Jul 16, 2026
@edwinjosechittilappilly
edwinjosechittilappilly merged commit d4f82e0 into main Jul 17, 2026
33 checks passed
@github-actions
github-actions Bot deleted the backport/2104-button-click-tracking branch July 17, 2026 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend 🟨 Issues related to the UI/UX lgtm refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants