Skip to content

Feat: Added google analytics tracking#627

Merged
shashilo merged 4 commits intodevelopfrom
alex/626-add-google-analytics
Oct 16, 2025
Merged

Feat: Added google analytics tracking#627
shashilo merged 4 commits intodevelopfrom
alex/626-add-google-analytics

Conversation

@alexappleget
Copy link
Collaborator

@alexappleget alexappleget commented Oct 5, 2025

Description

Before:

Didn't have google analytics tracking.

After:

Allows google analytics to track all web pages

Closes #626

Testing instructions

Not applicable

Additional information

No extra info to share

Pre-submission checklist

  • Code builds and passes locally
  • PR title follows Conventional Commit format (e.g. test #001: created unit test for __ component)
  • Request reviews from the Peer Code Reviewers and Senior+ Code Reviewers groups
  • Thread has been created in Discord and PR is linked in gis-code-questions

@vercel
Copy link

vercel bot commented Oct 5, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
elecretanta Ready Ready Preview Comment Oct 16, 2025 4:25pm
elecretanta-storybook Ready Ready Preview Comment Oct 16, 2025 4:25pm
elecretanta-unit-test Ready Ready Preview Comment Oct 16, 2025 4:25pm

shashilo
shashilo previously approved these changes Oct 7, 2025
<html lang="en">
<head>
<Script
src={`https://www.googletagmanager.com/gtag/js?id=${process.env.NEXT_PUBLIC_GOOGLE_ANALYTICS_ID}`}
Copy link
Collaborator

Choose a reason for hiding this comment

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

G-D4KQR9DSXY

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

added

Copy link
Member

@nickytonline nickytonline left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

<head>
<Script
src={`https://www.googletagmanager.com/gtag/js?id=${process.env.NEXT_PUBLIC_GOOGLE_ANALYTICS_ID}`}
strategy="afterInteractive"
Copy link
Member

Choose a reason for hiding this comment

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

For reference to anyone new to strategy="afterInteractive". Essentially this is Next.js setting the script as async for you.

lillian-nguyen
lillian-nguyen previously approved these changes Oct 14, 2025
nickytonline
nickytonline previously approved these changes Oct 14, 2025
@shashilo shashilo merged commit 0a2d352 into develop Oct 16, 2025
4 of 5 checks passed
@shashilo shashilo deleted the alex/626-add-google-analytics branch October 16, 2025 16:25
shashilo added a commit that referenced this pull request Oct 23, 2025
Co-authored-by: Shashi Lo <362527+shashilo@users.noreply.github.com>
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.

Add google analytics

4 participants