Skip to content

perf(agents): Prefetch and load drawer independently of trace's table#107663

Merged
priscilawebdev merged 3 commits intomasterfrom
pricila/perf/open-drawer-immediately-when-pages-loads
Feb 6, 2026
Merged

perf(agents): Prefetch and load drawer independently of trace's table#107663
priscilawebdev merged 3 commits intomasterfrom
pricila/perf/open-drawer-immediately-when-pages-loads

Conversation

@priscilawebdev
Copy link
Member

We’re happy with the result, so I am replicating the same solution used in #107655.

closes https://linear.app/getsentry/issue/TET-1880/do-not-wait-for-table-to-finish-loading-before-loading-conversation

@linear
Copy link

linear bot commented Feb 5, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Feb 5, 2026
@priscilawebdev priscilawebdev marked this pull request as ready for review February 5, 2026 11:13
@priscilawebdev priscilawebdev requested a review from a team as a code owner February 5, 2026 11:13
Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Extract URL-based drawer opening into a separate hook (useOpenTraceViewDrawerFromUrl)
that is called only once at the top level of AgentsOverviewPage. This prevents
duplicate trackAnalytics('agent-monitoring.drawer.open') events when multiple
components use useTraceViewDrawer.
Pass openTraceViewDrawer as a prop to TracesTable instead of calling
the hook inside it. This ensures only one hook instance exists,
preventing duplicate trackAnalytics('agent-monitoring.drawer.open')
events when the page loads with a trace URL.
@priscilawebdev priscilawebdev force-pushed the pricila/perf/open-drawer-immediately-when-pages-loads branch from 5042320 to c60b84a Compare February 5, 2026 12:30
@priscilawebdev priscilawebdev merged commit 1ddf874 into master Feb 6, 2026
59 checks passed
@priscilawebdev priscilawebdev deleted the pricila/perf/open-drawer-immediately-when-pages-loads branch February 6, 2026 06:18
jaydgoss pushed a commit that referenced this pull request Feb 12, 2026
…#107663)

We’re happy with the result, so I am replicating the same solution used
in #107655.
dcramer pushed a commit that referenced this pull request Feb 17, 2026
…#107663)

We’re happy with the result, so I am replicating the same solution used
in #107655.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants