-
Notifications
You must be signed in to change notification settings - Fork 567
posthog migration: part 3 #7365
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
5 Skipped Deployments
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the WalkthroughThis change removes all analytics tracking infrastructure and logic from the dashboard application. All tracking-related props, components (such as TrackedLink, TrackedLinkTW, TrackedUnderlineLink, TrackedCopyButton), and the useTrack hook are deleted. All UI components and links now use standard, non-tracking equivalents, and all tracking metadata is eliminated from props and types. Additionally, the Changes
Sequence Diagram(s)Not generated, as the changes are broad refactors removing tracking logic rather than introducing or modifying specific control flows. Possibly related PRs
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
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. |
@coderabbitai review |
✅ Actions performedReview triggered.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7365 +/- ##
=======================================
Coverage 52.35% 52.35%
=======================================
Files 939 939
Lines 63161 63161
Branches 4217 4217
=======================================
Hits 33070 33070
Misses 29984 29984
Partials 107 107
🚀 New features to boost your workflow:
|
d3a5396
to
f0bb043
Compare
604a6e9
to
6754c8b
Compare
f0bb043
to
50192f2
Compare
6754c8b
to
5924b03
Compare
Remove Analytics Tracking Components
This PR removes analytics tracking components and properties from the dashboard app. Key changes include:
TrackedLinkTW
andTrackedUnderlineLink
components, replacing them with standardLink
andUnderlineLink
componentsThese changes simplify the codebase by removing the analytics tracking layer while maintaining all core functionality.
PR-Codex overview
This PR focuses on enhancing link security by adding
rel="noopener noreferrer"
to various<Link>
components throughout the application, which helps prevent potential security risks when opening links in new tabs.Detailed summary
rel="noopener noreferrer"
to multiple<Link>
components.TrackedLinkTW
component in favor of standard<Link>
for consistency.Summary by CodeRabbit
Refactor
Style
rel="noopener noreferrer"
attribute to all external links opening in new tabs for improved security and privacy.No changes to core functionality or user workflows; navigation and UI remain consistent.