Skip to content

Conversation

suejung-sentry
Copy link
Member

[DO NOT MERGE] - for demo only

@suejung-sentry suejung-sentry requested a review from a team as a code owner October 8, 2025 19:50
@suejung-sentry suejung-sentry removed the request for review from a team October 8, 2025 19:50
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Oct 8, 2025
@suejung-sentry suejung-sentry changed the base branch from master to sshin/demo-base October 8, 2025 19:50
@suejung-sentry suejung-sentry changed the title feat(prevent): Add logaf sensitivity dropdown to ai config (#101040) feat(prevent): Add logaf sensitivity dropdown to ai config Oct 8, 2025
@suejung-sentry
Copy link
Member Author

@sentry generate-test

@suejung-sentry
Copy link
Member Author

@sentry review

Comment on lines +30 to +47
const sensitivityOptions: SensitivityOption[] = [
{value: 'low', label: 'Low', details: 'Post all potential issues for maximum breadth.'},
{
value: 'medium',
label: 'Medium',
details: 'Post likely issues for a balance of thoroughness and noise.',
},
{
value: 'high',
label: 'High',
details: 'Post only major issues to highlight most impactful findings.',
},
{
value: 'critical',
label: 'Critical',
details: 'Post only high-impact, high-sensitivity issues for maximum focus.',
},
];
Copy link
Contributor

Choose a reason for hiding this comment

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

The sensitivityOptions array contains hardcoded English strings for labels and details. Consider using the t() function for internationalization to ensure these strings can be translated. This is especially important for user-facing text that explains feature behavior.

Did we get this right? 👍 / 👎 to inform future reviews.

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.

1 participant