Skip to content

chore(analytics): update Analytics component to use development mode across apps - #1284

Open
felicio wants to merge 1 commit into
developfrom
felicio/vercel-analytics
Open

chore(analytics): update Analytics component to use development mode across apps#1284
felicio wants to merge 1 commit into
developfrom
felicio/vercel-analytics

Conversation

@felicio

@felicio felicio commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Copilot AI review requested due to automatic review settings July 22, 2026 07:56
@vercel

vercel Bot commented Jul 22, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
status-get-status-app Ready Ready Preview, Comment Jul 22, 2026 8:02am
status-network-website Ready Ready Preview, Comment Jul 22, 2026 8:02am
status-portfolio Ready Ready Preview, Comment Jul 22, 2026 8:02am
status-website Ready Ready Preview, Comment Jul 22, 2026 8:02am
4 Skipped Deployments
Project Deployment Actions Updated (UTC)
community-dapp Ignored Ignored Jul 22, 2026 8:02am
status-api Ignored Ignored Jul 22, 2026 8:02am
status-components Ignored Ignored Jul 22, 2026 8:02am
status-network-hub Ignored Ignored Jul 22, 2026 8:02am

Request Review

@changeset-bot

changeset-bot Bot commented Jul 22, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 08c59e7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 standardizes Vercel Analytics initialization across multiple Next.js apps in the monorepo by explicitly rendering the @vercel/analytics/next <Analytics /> component in "development" mode.

Changes:

  • Updated <Analytics /> usage to mode="development" across apps.
  • Consolidated the duplicate <Analytics /> renders in apps/status.app into a single component instance.
  • Kept existing Umami script injection unchanged while adjusting Vercel Analytics configuration.

Reviewed changes

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

File Description
apps/status.network/src/app/[locale]/layout.tsx Sets Vercel <Analytics /> to mode="development" in the locale root layout.
apps/status.app/src/app/layout.tsx Replaces two <Analytics /> instances with a single one configured for development mode.
apps/portfolio/src/app/layout.tsx Sets Vercel <Analytics /> to mode="development" in the root layout.
apps/get.status.app/src/app/layout.tsx Sets Vercel <Analytics /> to mode="development" in the root layout.

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

</div>
<ToastContainer />
<Analytics debug={false} />
<Analytics mode="development" debug={false} />

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.

@felicio Do we need to set mode to "production" for the live website?

Ref: https://vercel.com/docs/analytics/package#mode

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Thanks @jinhojang6. Since the production has been moved from Vercel to Jenkins, it would not be necessary.

@JulesFiliot JulesFiliot 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.

From what I understand having mode set to development and debug to false blocks all analytics from being sent on any environment. In that case why not just remove all <Analytics /> tags to avoid dead code?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

4 participants