Skip to content

Fix review opportunity filtering#7155

Merged
jmgasper merged 1 commit into
masterfrom
develop
Dec 3, 2025
Merged

Fix review opportunity filtering#7155
jmgasper merged 1 commit into
masterfrom
develop

Conversation

@jmgasper

@jmgasper jmgasper commented Dec 3, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

@jmgasper jmgasper requested a review from kkartunov as a code owner December 3, 2025 23:01
// opp.challenge.track === 'QA' ? 'Dev' : trackAbbr[opp.challenge.track],
// ]),
track: trackAbbr[challengeData.track],
track: normalizedTrackMap[(challengeData.track || '').toString().trim().toUpperCase()]

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[❗❗ correctness]
The use of toString().trim().toUpperCase() for normalizing challengeData.track could lead to unexpected behavior if challengeData.track is null or undefined. Consider using optional chaining or a default value to ensure safety.

@jmgasper jmgasper merged commit 8d0f554 into master Dec 3, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant