Skip to content

fix: GA setup for Register Now and add Sponsor's website link to the logo#53

Merged
theamankumarsingh merged 2 commits intomainfrom
develop
Mar 19, 2026
Merged

fix: GA setup for Register Now and add Sponsor's website link to the logo#53
theamankumarsingh merged 2 commits intomainfrom
develop

Conversation

@theamankumarsingh
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR centralizes Google Analytics tracking for “Register” CTA clicks and updates landing-page sponsor logos to link out to sponsor websites.

Changes:

  • Added a shared trackRegisterClick utility to emit a register_cta_click GA event and then navigate.
  • Replaced inline gtag click handlers in TopBar, MobileMenu, Header, and CTA section with the shared tracker.
  • Wrapped sponsor logos in the landing header with outbound links (TCS, NIQ).

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
src/utils/trackRegisterClick.ts Introduces shared GA click tracking + delayed navigation logic.
src/components/shared/TopBar/index.tsx Uses the shared tracker for desktop + mobile top-bar “Register” CTAs.
src/components/shared/TopBar/MobileMenu.tsx Uses the shared tracker for the mobile nav CTA and adds external-link attributes.
src/components/landing/Header.tsx Uses the shared tracker for the header Register button; adds sponsor logo outbound links.
src/components/landing/CTAReg.tsx Uses the shared tracker for the CTA section Register button and adds external-link attributes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@theamankumarsingh theamankumarsingh merged commit 6014fe7 into main Mar 19, 2026
6 checks passed
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