Skip to content

Conversation

@cursor
Copy link
Contributor

@cursor cursor bot commented Dec 11, 2025

Updates the NoIssuesMatched component to improve user guidance when no issues are found.

This PR:

  • Updates the second bullet point to directly link "right syntax" to the search documentation.
  • Adds a new line suggesting users explore raw telemetry (logs) if they are still not seeing issues.

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.


Slack Thread

Open in Cursor Open in Web

@cursor
Copy link
Contributor Author

cursor bot commented Dec 11, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 11, 2025
@mrduncan
Copy link
Member

dev-ui:
Screenshot 2025-12-12 at 1 13 27 PM

Confirmed links are good as well

@mrduncan mrduncan marked this pull request as ready for review December 12, 2025 21:17
@mrduncan mrduncan requested a review from a team as a code owner December 12, 2025 21:17
),
}
)}
</div>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Bug: Logs link shown without checking feature flag

The new logs link is displayed unconditionally to all users, but the logs feature (explore/logs) is gated behind the ourlogs-enabled feature flag throughout the codebase. Other components like the command palette check organization.features.includes('ourlogs-enabled') before showing logs links. Users without this feature enabled will see a suggestion to explore logs but may encounter an error or inaccessible page when clicking the link. The organization from useOrganization() is already available in this component and could be used to conditionally render this section.

Fix in Cursor Fix in Web

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.

3 participants