Skip to content
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

[RAC][Security Solution] Alert table: Resolver and Cases icons to bulk action menu #108420

Merged
merged 37 commits into from
Aug 17, 2021

Conversation

stephmilovic
Copy link
Contributor

@stephmilovic stephmilovic commented Aug 12, 2021

Summary

Resolves #107917

Resolver and Cases alert table icons moved to ... overflow menu. Disabled action items are hidden.

Before:
Screen Shot 2021-08-12 at 2 32 02 PM

After
in alerts table:
Screen Shot 2021-08-12 at 12 32 06 PM

in timeline:
Screen Shot 2021-08-12 at 12 31 47 PM

in events viewer. this hasnt changed, but once those 4 top icons in the header move we will need to check the sizing of the column in this instance
Screen Shot 2021-08-12 at 12 27 34 PM

@stephmilovic stephmilovic added WIP Work in progress v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting Security Solution Threat Hunting Team Theme: rac label obsolete Feature:RAC label obsolete v7.15.0 labels Aug 12, 2021
@stephmilovic stephmilovic changed the title [RAC][Security Solution] [skip-ci] alert table icons to overflow menu [RAC][Security Solution] Alert table: Resolver and Cases icons to bulk action menu Aug 12, 2021
@stephmilovic stephmilovic marked this pull request as ready for review August 12, 2021 20:33
@stephmilovic stephmilovic requested a review from a team as a code owner August 12, 2021 20:33
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-threat-hunting (Team:Threat Hunting)

@stephmilovic
Copy link
Contributor Author

stephmilovic commented Aug 13, 2021

@elasticmachine merge upstream

@@ -205,7 +206,7 @@ export const TakeActionDropdown = React.memo(
ecsRowData: ecsData,
useInsertTimeline: insertTimelineHook,
casePermissions,
appId: 'securitySolution',
appId: APP_ID,
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@angorayc angorayc left a comment

Choose a reason for hiding this comment

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

Tested locally and all worked as expected. Good to go after rebasing with master, thank you Steph!

@@ -134,41 +134,6 @@ describe('EventColumnView', () => {
expect(wrapper.find('[data-test-subj="pin"]').exists()).toBe(false);
});

test('it render AddToCaseAction if timelineId === TimelineId.detectionsPage', () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

moved to .../detections/components/alerts_table/timeline_actions/alert_context_menu.test.tsx

@@ -246,56 +245,6 @@ describe('helpers', () => {
});
});

describe('isInvestigateInResolverActionEnabled', () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

moved to .../detections/components/alerts_table/timeline_actions/investigate_in_resolver.test.tsx

@@ -129,51 +123,6 @@ export const getEventType = (event: Ecs): Omit<TimelineEventsType, 'all'> => {
return 'raw';
};

export const isInvestigateInResolverActionEnabled = (ecsData?: Ecs) =>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

moved to .../detections/components/alerts_table/timeline_actions/investigate_in_resolver.tsx

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
securitySolution 2366 2364 -2

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 6.5MB 6.5MB -1.2KB
timelines 407.9KB 408.0KB +64.0B
total -1.1KB

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:RAC label obsolete release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting Security Solution Threat Hunting Team Theme: rac label obsolete v7.15.0 v8.0.0 WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Security Solution][RAC] - Table Functionality Updates
5 participants