Skip to content

Conversation

@MananTank
Copy link
Member

@MananTank MananTank commented Mar 31, 2025

PR-Codex overview

This PR introduces enhancements to the PostHog integration across multiple applications, including the addition of the PosthogHeadSetup and PHProvider components, updates to environment variable handling, and adjustments in the next.config.mjs for improved security and functionality.

Detailed summary

  • Added isProd constant in env.ts to check production environment.
  • Introduced PosthogHeadSetup component for preconnecting to PostHog.
  • Created PHProvider to manage PostHog initialization and context.
  • Updated next.config.mjs to refine script-src directives.
  • Implemented PosthogPageView component for tracking page views in multiple apps.
  • Enhanced PosthogIdentifier to ensure PostHog is loaded before capturing events.
  • Removed the old PostHogProvider from layout files and replaced it with PHProvider.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@linear
Copy link

linear bot commented Mar 31, 2025

@vercel
Copy link

vercel bot commented Mar 31, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 31, 2025 6:47pm
login ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 31, 2025 6:47pm
thirdweb_playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 31, 2025 6:47pm
thirdweb-www ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 31, 2025 6:47pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
wallet-ui ⬜️ Skipped (Inspect) Mar 31, 2025 6:47pm

@vercel vercel bot temporarily deployed to Preview – wallet-ui March 31, 2025 18:19 Inactive
@changeset-bot
Copy link

changeset-bot bot commented Mar 31, 2025

⚠️ No Changeset found

Latest commit: a17d893

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

@github-actions github-actions bot added Dashboard Involves changes to the Dashboard. Playground Changes involving the Playground codebase. Portal Involves changes to the Portal (docs) codebase. labels Mar 31, 2025
Copy link
Member Author


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • merge-queue - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@MananTank MananTank marked this pull request as ready for review March 31, 2025 18:19
@MananTank MananTank requested review from a team as code owners March 31, 2025 18:19
@MananTank MananTank changed the title [TOOL-3873] Fix pageview posthog tracking in dashboard, portal. Add posthog to playground [TOOL-3873] Fix pageview posthog tracking in dashboard. Add posthog to playground Mar 31, 2025
@codecov
Copy link

codecov bot commented Mar 31, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 54.98%. Comparing base (0dcde07) to head (a17d893).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6595   +/-   ##
=======================================
  Coverage   54.98%   54.98%           
=======================================
  Files         885      885           
  Lines       55517    55517           
  Branches     3794     3794           
=======================================
  Hits        30528    30528           
  Misses      24894    24894           
  Partials       95       95           
Flag Coverage Δ
packages 54.98% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@MananTank MananTank changed the title [TOOL-3873] Fix pageview posthog tracking in dashboard. Add posthog to playground [TOOL-3873] Fix pageview posthog tracking in dashboard. Add posthog to playground, include searchParams in pageview tracking in portal Mar 31, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Mar 31, 2025

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
thirdweb (esm) 47.56 KB (0%) 952 ms (0%) 171 ms (+94.82% 🔺) 1.2 s
thirdweb (cjs) 127.31 KB (0%) 2.6 s (0%) 285 ms (+12.11% 🔺) 2.9 s
thirdweb (minimal + tree-shaking) 5.6 KB (0%) 113 ms (0%) 71 ms (+561.97% 🔺) 183 ms
thirdweb/chains (tree-shaking) 506 B (0%) 10 ms (0%) 50 ms (+1525.91% 🔺) 60 ms
thirdweb/react (minimal + tree-shaking) 19.34 KB (0%) 387 ms (0%) 108 ms (+435.41% 🔺) 495 ms

@MananTank MananTank changed the title [TOOL-3873] Fix pageview posthog tracking in dashboard. Add posthog to playground, include searchParams in pageview tracking in portal [TOOL-3873] Fix pageview tracking in dashboard, Add posthog to playground Mar 31, 2025
@graphite-app
Copy link
Contributor

graphite-app bot commented Mar 31, 2025

Merge activity

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

Labels

Dashboard Involves changes to the Dashboard. Playground Changes involving the Playground codebase. Portal Involves changes to the Portal (docs) codebase.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants