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

[7.x] [Security Solution][Case] Case action type (#80870) #82584

Merged
merged 1 commit into from
Nov 4, 2020

Conversation

cnasikas
Copy link
Member

@cnasikas cnasikas commented Nov 4, 2020

Backports the following commits to 7.x:

* Init connector

* Add test

* Improve comment type

* Add integration tests

* Fix i18n

* Improve tests

* Show unknown when username is null

* Improve comment type

* Pass connector to case client

* Improve type after PR elastic#82125

* Add comment migration test

* Fix integration tests

* Fix reporter on table

* Create case connector ui

* Add connector to README

* Improve casting on executor

* Translate name

* Improve test

* Create comment type enum

* Fix type

* Fix i18n

* Move README to cases

* Filter out case connector from alerting

Co-authored-by: Mike Côté <[email protected]>

Co-authored-by: Mike Côté <[email protected]>
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

@kbn/optimizer bundle module count

id before after diff
securitySolution 2064 2067 +3

async chunks size

id before after diff
securitySolution 7.8MB 7.8MB -36.0B
triggersActionsUi 1.5MB 1.5MB +127.0B
total +91.0B

distributable file count

id before after diff
default 43062 43068 +6

page load bundle size

id before after diff
securitySolution 242.1KB 243.0KB +963.0B
triggersActionsUi 132.5KB 132.7KB +166.0B
total +1.1KB

Saved Objects .kibana field count

id before after diff
cases-comments 17 18 +1

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

@cnasikas cnasikas merged commit ac0f704 into elastic:7.x Nov 4, 2020
@cnasikas cnasikas deleted the backport/7.x/pr-80870 branch November 4, 2020 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants