Introduce Posthog #960
Merged
Introduce Posthog #960
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Deploying ens-app-v3 with
|
| Latest commit: |
27b7ad6
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://7837b541.ens-app-v3.pages.dev |
| Branch Preview URL: | https://introduce-posthog.ens-app-v3.pages.dev |
b9556cf to
674d307
Compare
Introduces comprehensive analytics tracking using PostHog across the application: - Added PostHog configuration and provider - Created type-safe event tracking utility - Implemented analytics events for key user interactions in registration, import, search, and wallet flows - Added environment variables for PostHog configuration - Removed legacy event tracking hooks
- Add chain ID registration to PostHog tracking - Enhance user identification with initial address property
- Modify event name for DotBox registration tracking - Add duration type to registration event tracking
- Modify console listener regex to match PostHog event tracking - Update event name assertion from 'search_selected_eth' to 'search:select' - Import EventDefs from analytics events utility
- Standardize event tracking names across import, registration, and box registration specs - Update console listener regex to match PostHog event tracking - Simplify and normalize event names (e.g., 'search:select', 'register:pricing', 'wallet:open')
- Add production detection function to conditionally log metrics events - Update PostHog provider and event tracking to use production detection - Modify console logging to use consistent '[Metrics]' prefix - Simplify event logging across different components
Update CSP to allow scripts from PostHog analytics domain, complementing recent analytics integration
Update registration test to allow for gas percentage fluctuations by using a more flexible regex match
- Change PostHog host to custom domain - Update Content Security Policy to allow scripts from new PostHog host - Modify default PostHog API host to EU endpoint
- Implement cookie consent system with localStorage persistence - Create CookieConsentBanner component for user consent - Add cookie consent settings to Privacy section - Configure PostHog persistence based on user consent
Optimize pattern matching in PrivacySection to use `.otherwise()` for handling undecided cookie consent state
Remove unnecessary default export from CookieConsentBanner component to maintain consistency with existing import patterns
f83f775 to
9c54e15
Compare
LeonmanRolls
approved these changes
Mar 12, 2025
Add state tracking for banner visibility to ensure consistent display of cookie consent banner when consent is undecided
Introduce useSetupIntercom hook to centralize Intercom initialization and tracking, removing direct Intercom configuration from Basic layout and improving separation of concerns
LeonmanRolls
approved these changes
Mar 13, 2025
Refactor event names in the registration flow to follow a consistent naming convention. Introduce new tracking events for transaction states, including sending, failure, and completion. Remove deprecated event names to streamline analytics integration.
Move the tracking event test for 'transaction:register:send' to a new position within the registration steps to improve clarity and maintainability of the test suite.
Eliminate the unnecessary call to clear console messages in the tracking event test for 'wallet:open' to streamline the test execution and improve clarity.
Move the tracking event test for 'transaction:register:send' back into the registration flow to enhance clarity and ensure proper event tracking during the registration process.
Introduce new tracking events for import transaction states (send, complete, fail) and refactor existing tests to ensure accurate event firing during the import process. Remove deprecated event names to streamline analytics integration.
Refactor the test for DNS import tracking to use the updated event name 'transaction:import:send' instead of 'import:transaction_start' to ensure consistency with the latest event naming conventions.
Change the event name in the registration test from 'register:transaction_override' to 'transaction:register:override' to align with the updated naming conventions and ensure accurate event tracking.
Reintroduce the gas cost check in the disabled condition of the import next button to prevent user interaction when gas cost is zero, ensuring proper flow control during the import process.
|
sugh01
approved these changes
Mar 21, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.