Skip to content

Conversation

@aditya520
Copy link
Member

Summary

Rationale

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

@aditya520 aditya520 requested a review from a team as a code owner November 27, 2025 16:43
@vercel
Copy link

vercel bot commented Nov 27, 2025

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

Project Deployment Preview Comments Updated (UTC)
component-library Ready Ready Preview Comment Nov 27, 2025 5:23pm
developer-hub Ready Ready Preview Comment Nov 27, 2025 5:23pm
5 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
api-reference Skipped Skipped Nov 27, 2025 5:23pm
entropy-explorer Skipped Skipped Nov 27, 2025 5:23pm
insights Skipped Skipped Nov 27, 2025 5:23pm
proposals Skipped Skipped Nov 27, 2025 5:23pm
staking Skipped Skipped Nov 27, 2025 5:23pm

return <Root afterBodyContent={renderGoogleAnalytics()}>{children}</Root>;
}

function renderGoogleAnalytics(): ReactNode | undefined {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use <GoogleAnalytics> from @next/third-parties/google for this, see e.g. https://github.com/pyth-network/pyth-crosschain/blob/main/packages/component-library/src/AppShell/index.tsx#L55

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh yes, that looks better.

@aditya520 aditya520 merged commit 9da74c6 into main Nov 27, 2025
10 checks passed
@aditya520 aditya520 deleted the chore(dev-hub)-google-analytics branch November 27, 2025 17:39
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.

3 participants