-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
feat(prevent): Add logaf sensitivity dropdown to ai config #101187
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: sshin/demo-base
Are you sure you want to change the base?
Conversation
@sentry generate-test |
c53a131
to
e92df48
Compare
@sentry review |
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.', | ||
}, | ||
]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The sensitivity options contain hardcoded English strings that should be internationalized using the t()
function for proper localization support. This is especially important for user-facing text like labels and descriptions.
Did we get this right? 👍 / 👎 to inform future reviews.
[DO NOT MERGE] - For demo only