Skip to content

feat(preprod): Add preprod issue types to frontend#104600

Merged
chromy merged 1 commit intomasterfrom
chromy/2025-12-09-issues-v2-part02
Dec 11, 2025
Merged

feat(preprod): Add preprod issue types to frontend#104600
chromy merged 1 commit intomasterfrom
chromy/2025-12-09-issues-v2-part02

Conversation

@chromy
Copy link
Copy Markdown
Contributor

@chromy chromy commented Dec 9, 2025

No description provided.

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 9, 2025
Comment on lines 106 to 112
[IssueCategory.MOBILE]: mobileConfig,
[IssueCategory.METRIC]: metricConfig,
[IssueCategory.AI_DETECTED]: aiDetectedConfig,
[IssueCategory.PREPROD]: preprodConfig,
};

/**

This comment was marked as outdated.

[IssueCategory.MOBILE]: mobileConfig,
[IssueCategory.METRIC]: metricConfig,
[IssueCategory.AI_DETECTED]: aiDetectedConfig,
[IssueCategory.PREPROD]: preprodConfig,
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.

Bug: PREPROD occurrence types misrouted to PERFORMANCE category

The preprodConfig is registered at line 109, but the eventOccurrenceTypeToIssueCategory function (lines 132-137) was not updated to handle PREPROD occurrence types. Since 11001 and 11002 are >= 1000, they will be incorrectly classified as IssueCategory.PERFORMANCE instead of IssueCategory.PREPROD when config is looked up via eventOccurrenceType. This causes the PREPROD config to never be used when no group object is available for the occurrence type-based lookup path.

Fix in Cursor Fix in Web

@chromy chromy merged commit 472b283 into master Dec 11, 2025
49 checks passed
@chromy chromy deleted the chromy/2025-12-09-issues-v2-part02 branch December 11, 2025 21:10
@github-actions github-actions bot locked and limited conversation to collaborators Dec 27, 2025
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