-
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
[ML] [SIEM] update links #48123
[ML] [SIEM] update links #48123
Conversation
Pinging @elastic/siem (Team:SIEM) |
💚 Build Succeeded |
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.
Could you rebase this against master? This is only picking up files from the first two SIEM modules, before the module names and jobs were edited in #47848. Looking at the latest code, 17 jobs contain custom URLs with kqlQuery
, but this is only editing 14 jobs.
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.
Edits LGTM. Note I couldn't actually test the edits to the custom URLs as this PR is dependent on #47306 which is yet to be merged.
💔 Build Failed |
💚 Build Succeeded |
💚 Build Succeeded |
Summary
Update SIEM links to match new pardigm PR#47306. Let's make sure to merge this one when this PR#47306 is merged
Checklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.[ ] This was checked for cross-browser compatibility, including a check against IE11[ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support[ ] Documentation was added for features that require explanation or tutorials[ ] Unit or functional tests were updated or added to match the most common scenarios[ ] This was checked for keyboard-only and screenreader accessibilityFor maintainers
[ ] This was checked for breaking API changes and was labeled appropriately[ ] This includes a feature addition or change that requires a release note and was labeled appropriately