Skip to content

Fixed #17329 Audit Warning Threshold could be negative#18116

Merged
snipe merged 2 commits intogrokability:developfrom
akemidx:auditwarningthreshold
Oct 28, 2025
Merged

Fixed #17329 Audit Warning Threshold could be negative#18116
snipe merged 2 commits intogrokability:developfrom
akemidx:auditwarningthreshold

Conversation

@akemidx
Copy link
Copy Markdown
Contributor

@akemidx akemidx commented Oct 28, 2025

Audit Warning Threshold could be negative.
This fixes the field so it must be greater than or equal to 0.
The field can still be null.

Screenshot 2025-10-28 at 16 18 19

Fixes #17329

@akemidx akemidx requested a review from snipe as a code owner October 28, 2025 20:20
Copy link
Copy Markdown
Member

@snipe snipe left a comment

Choose a reason for hiding this comment

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

You'll want to add min="0" to the HTML input field as well, to prevent the numeric stepper from allowing a negative.

@akemidx
Copy link
Copy Markdown
Contributor Author

akemidx commented Oct 28, 2025

You'll want to add min="0" to the HTML input field as well, to prevent the numeric stepper from allowing a negative.

yea i was wondering about that. but the field already was catching the input.
lemme fix rq.

@snipe
Copy link
Copy Markdown
Member

snipe commented Oct 28, 2025

What res are you looking at that screen on? I'm curious why the layout looks so odd in your screenshot (right-aligned, and breaking down to the next line.)

@akemidx
Copy link
Copy Markdown
Contributor Author

akemidx commented Oct 28, 2025

Screenshot 2025-10-28 at 16 37 43

I usually do a lot of my coding with console open. here is it without.

The op is at about 600x950

@snipe
Copy link
Copy Markdown
Member

snipe commented Oct 28, 2025

Right, but on a smaller screen, it shouldn't be behaving like that (aligned right). I'll take a look.

@snipe snipe merged commit 8d0c88d into grokability:develop Oct 28, 2025
6 of 7 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.

2 participants