-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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][Exceptions] - Create/ refactor Exception-List common's components in @kbn/securitysolution-exception-list-components #143363
[Security Solution][Exceptions] - Create/ refactor Exception-List common's components in @kbn/securitysolution-exception-list-components #143363
Conversation
@elasticmachine merge upstream |
securityLinkAnchorComponent: ElementType; // This property needs to be removed to avoid the Prop Drilling, once we move all the common components from x-pack/security-solution/common | ||
} | ||
|
||
export const generateLinedRulesMenuItems = ({ |
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.
export const generateLinedRulesMenuItems = ({ | |
export const generateLinkedRulesMenuItems = ({ |
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.
Thanks [f015511]
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.
overall looks good to me. Preference is to merge this after yara's PR #143127 if that's okay!
…Nasr/kibana into common-list-components-pkg
We decided to merge this PR as it is not related to Yara-PR's files |
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Public APIs missing comments
Unknown metric groupsAPI count
History
To update your PR or re-run it, just comment with: cc @WafaaNasr |
…mon's components in @kbn/securitysolution-exception-list-components (elastic#143363) * feat: add list header components + refactoring * add tests for comments and conditions components * remove unused var * complete tests for exception_item_card, excpetion_items, empty_viewer_state * add test for useExceptionItemCard hook * add tests for generateLinedRulesMenuItems * add readme and index.md * Update index.md * remove unused file * remove unused file * add tests for Header_menu * extract security mocks to a file * test for header * add missing tests * fix tests * fix text_with_edit dataTestSubj * apply rewview comment Co-authored-by: Kibana Machine <[email protected]>
Pinging @elastic/security-solution (Team: SecuritySolution) |
Exception List component in @kbn/securitysolution-exception-list-components package's changes summary
Continuing from #140985
1. List Header
2. Edit Modal
3. Header Menu
, ,
4. Search Bar
5. Exception Item Card
6. Exception Items
7. Pagination
8. EmptyViewerState
a. Empty Search
b. Empty
c. Error
Checklist