Skip to content

Conversation

robearlam
Copy link
Member

Added GA Tracking for the SUGCON Sites, powered by env var.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have read the Contributing guide.
  • My code/comments/docs fully adhere to the Code of Conduct.
  • My change is a code change.
  • My change is a documentation change and there are NO other updates required.

@robearlam robearlam requested a review from Copilot July 16, 2025 03:10
Copy link
Contributor

@Copilot 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 adds Google Analytics tracking functionality to the SUGCON 2024 site using Next.js third-party integration. The implementation is controlled by an environment variable to enable flexible configuration across different environments.

  • Integrates Google Analytics using the @next/third-parties/google package
  • Conditionally renders Google Analytics based on the NEXT_PUBLIC_GTAG environment variable
  • Updates package dependencies to include the required third-party integration package

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
headapps/Sugcon2024/src/Layout.tsx Adds Google Analytics component with conditional rendering based on environment variable
headapps/Sugcon2024/package.json Adds @next/third-parties dependency for Google Analytics integration
Files not reviewed (1)
  • headapps/Sugcon2024/package-lock.json: Language not supported
Comments suppressed due to low confidence (3)

@robearlam robearlam merged commit 006073a into main Jul 16, 2025
7 checks passed
@robearlam robearlam deleted the feat/ga-tracking branch July 16, 2025 04:25
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.

1 participant