Skip to content

fix(integrations): report matching result count in search analytics#119649

Merged
cvxluo merged 1 commit into
masterfrom
cvxluo/report-matching-result-count-in-directory-search-a
Jul 21, 2026
Merged

fix(integrations): report matching result count in search analytics#119649
cvxluo merged 1 commit into
masterfrom
cvxluo/report-matching-result-count-in-directory-search-a

Conversation

@cvxluo

@cvxluo cvxluo commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Before, this analytics event would always report num_results as 100ish, since it just reported the whole list of integrations reported by the API. This reports the number of results after searches are actually done.

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jul 14, 2026
@cvxluo
cvxluo marked this pull request as ready for review July 14, 2026 18:02
@cvxluo
cvxluo requested a review from a team as a code owner July 14, 2026 18:02
@cvxluo
cvxluo requested a review from a team July 14, 2026 18:02
Comment thread static/app/views/settings/organizationIntegrations/integrationListDirectory.tsx Outdated
@cvxluo
cvxluo force-pushed the cvxluo/report-matching-result-count-in-directory-search-a branch from 8e064d6 to 88fb28e Compare July 14, 2026 18:16

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

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.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 88fb28e. Configure here.

@cvxluo
cvxluo force-pushed the cvxluo/report-matching-result-count-in-directory-search-a branch from 88fb28e to cfea5e1 Compare July 15, 2026 18:49
…nalytics

Co-authored-by: Claude <noreply@anthropic.com>
@cvxluo
cvxluo force-pushed the cvxluo/report-matching-result-count-in-directory-search-a branch from cfea5e1 to 40a0bd8 Compare July 20, 2026 18:24
@cvxluo
cvxluo merged commit 10fa210 into master Jul 21, 2026
63 checks passed
@cvxluo
cvxluo deleted the cvxluo/report-matching-result-count-in-directory-search-a branch July 21, 2026 16:41
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