-
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
[Cloud Security] Host Name Misconfiguration Datagrid & Refactor CSP Plugin PHASE 1 #192114
[Cloud Security] Host Name Misconfiguration Datagrid & Refactor CSP Plugin PHASE 1 #192114
Conversation
…xt to do will be to remove all usage of the added constants and function on csp plugins and update the import to be from the package
/ci |
/ci |
Pinging @elastic/kibana-cloud-security-posture (Team:Cloud Security) |
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 from Entity Analytics
/ci |
1 similar comment
/ci |
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, couple of small comments
@@ -6,14 +6,15 @@ | |||
*/ | |||
|
|||
import { renderHook, act } from '@testing-library/react-hooks/dom'; | |||
import { useNavigateFindings, useNavigateVulnerabilities } from './use_navigate_findings'; | |||
import { useNavigateVulnerabilities } from './use_navigate_findings'; | |||
import { useNavigateFindings } from './use_navigate_findings'; |
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.
seems like the change is done by mistake, two imports from the same file
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.
Woops, you are right
@@ -38,8 +38,6 @@ export const DETECTION_RULE_RULES_API_CURRENT_VERSION = '2023-10-31'; | |||
export const CLOUD_SECURITY_POSTURE_PACKAGE_NAME = 'cloud_security_posture'; | |||
|
|||
export const CDR_MISCONFIGURATIONS_DATA_VIEW_NAME = 'Latest Cloud Security Misconfigurations'; |
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.
even if we don't need this const for shared code, I'd move it too to avoid confusion, as CDR related constants belong together
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.
good call
💚 Build Succeeded
Metrics [docs]Module Count
Public APIs missing comments
Any counts in public APIs
Async chunks
Page load bundle
Unknown metric groupsAPI count
ESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: |
…lugin PHASE 1 (elastic#192114) ## Summary In an attempt to make Reviewing easier and more accurate, the implementation of Misconfiguration Data grid on Host.name flyout in Alerts Page will be split into 2 Phases Phase 1: Move Functions, Utils or Helpers, Hooks, constants to Package Phase 2: Implementing the feature This is Phase 1 of the process This PR also include a small bug fix mentioned here elastic#191677 (review) --------- Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Summary
In an attempt to make Reviewing easier and more accurate, the implementation of Misconfiguration Data grid on Host.name flyout in Alerts Page will be split into 2 Phases
Phase 1: Move Functions, Utils or Helpers, Hooks, constants to Package
Phase 2: Implementing the feature
This is Phase 1 of the process
This PR also include a small bug fix mentioned here #191677 (review)