Skip to content

feat(issues): Add stack trace, tags to top issues#105466

Merged
scttcper merged 2 commits intomasterfrom
scttcper/add-tags-stack-trace
Dec 26, 2025
Merged

feat(issues): Add stack trace, tags to top issues#105466
scttcper merged 2 commits intomasterfrom
scttcper/add-tags-stack-trace

Conversation

@scttcper
Copy link
Copy Markdown
Member

adds tags and an example stack trace to the single card layout. Currently you would need to click view all issues -> pick issue to see a stack trace or tags. I think adding a bit of aggregate data helps someone other than seer decide what the fix looks like

image

adds tags and an example stack trace to the single card layout. Currently you would need to click view all issues -> pick issue to see a stack trace or tags. I think adding a bit of aggregate data helps someone other than seer decide what the fix looks like
@scttcper scttcper requested a review from cvxluo December 26, 2025 20:46
@scttcper scttcper requested a review from a team as a code owner December 26, 2025 20:46
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 26, 2025
function useClusterEventsStats(groupIds: number[], statsPeriod: StatsPeriod) {
const organization = useOrganization();

const interval = statsPeriod === '7d' ? '1d' : '1d';
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.

Redundant ternary always returns same interval value

The ternary expression statsPeriod === '7d' ? '1d' : '1d' always evaluates to '1d' regardless of the condition. Both branches return the same value, making the conditional pointless. This appears to be incomplete code where different intervals were likely intended for different stat periods (e.g., '6h' for 7-day views vs '1d' for 30-day views). Currently only '30d' is passed to this function, so there's no immediate production impact, but this would cause unexpected behavior if '7d' support is added later.

Fix in Cursor Fix in Web

Copy link
Copy Markdown
Contributor

@cvxluo cvxluo left a comment

Choose a reason for hiding this comment

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

nice

@scttcper scttcper enabled auto-merge (squash) December 26, 2025 21:42
@scttcper scttcper merged commit e09dedf into master Dec 26, 2025
51 checks passed
@scttcper scttcper deleted the scttcper/add-tags-stack-trace branch December 26, 2025 21:50
@github-actions github-actions bot locked and limited conversation to collaborators Jan 11, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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