Skip to content

Conversation

@evanpurkhiser
Copy link
Member

No description provided.

@evanpurkhiser evanpurkhiser requested a review from a team October 30, 2025 17:40
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Oct 30, 2025
// Find any incident with an update that includes any of the componentFilter.
inc.incident_updates.some(update =>
update.affected_components?.some(c => componentFilter.includes(c.code))
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Bug: Component Filter Bypasses Resolved Status

The component filter applies to the original incidents array instead of the already-filtered filteredIncidents. When both !includeResolved and componentFilter are active, resolved incidents may incorrectly appear in the results, as the resolved status filtering is bypassed.

Fix in Cursor Fix in Web

@evanpurkhiser evanpurkhiser merged commit 56c2a93 into master Oct 31, 2025
48 checks passed
@evanpurkhiser evanpurkhiser deleted the evanpurkhiser/fix-service-incidents-find-incidents-with-components-in-updates branch October 31, 2025 17:01
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