-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Security Solution][Detections] Add empty string validation for Tags and Authors #101756
[Security Solution][Detections] Add empty string validation for Tags and Authors #101756
Conversation
f5e3d13
to
fbd3ace
Compare
Pinging @elastic/security-solution (Team: SecuritySolution) |
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
x-pack/plugins/security_solution/public/detections/components/rules/step_about_rule/schema.tsx
Outdated
Show resolved
Hide resolved
35e4279
to
7cdf967
Compare
This PR is waiting for #102338 to be closed. After that, the fix will be a one-liner: { validator: emptyField(...), type: VALIDATION_TYPES.ARRAY_ITEM } |
7cdf967
to
e283a86
Compare
e283a86
to
22289e6
Compare
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.
LGTM! Thank you for your patience!
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
💚 Build SucceededMetrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: cc @machadoum |
Summary
issue: #91821
Adds a validation that doesn't allow empty strings for Tags and Authors.
UI when the user tries to add and empty string (" ")
Checklist
Delete any items that are not applicable to this PR.