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

Fix/cases connector deprecated #132236

Closed
wants to merge 76 commits into from

Conversation

academo
Copy link
Contributor

@academo academo commented May 16, 2022

Summary

Changes the logic to mark a connectors a deprecated by using the backend "isDeprecated" attribute. This change reflects in all places in cases where a connector can be selected.

Closes: #131033

image
image
image

Esteban Beltran and others added 30 commits April 26, 2022 11:35
* [Discover] Unskip graph security flaky tests

* [Discover] Tmp to check at CI

* [Discover] Revert tmp check

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
brianseeders and others added 27 commits May 4, 2022 09:40
…131232)

* Add fix license link

* fix localization

* fix CI error

* fix more translation issues

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
* rule state filter

* turn off experiment

* [CI] Auto-commit changed files from 'node scripts/eslint --no-cache --fix'

* Status filter API call

* Fix tests

* rename state to status, added tests

* Address comments and fix tests

* Revert experiment flag

* Remove unused translations

* Addressed comments

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
* [storybook] Watch for changes in packages

* Update default_config.ts
Co-authored-by: Lisa Cawley <lcawley@elastic.co>
@academo academo self-assigned this May 16, 2022
@academo academo closed this May 16, 2022
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
cases 427 428 +1
securitySolution 2920 2921 +1
total +2

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
actions 191 192 +1

Async chunks

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

id before after diff
observability 436.2KB 436.2KB +8.0B
securitySolution 5.0MB 5.0MB +931.0B
timelines 26.1KB 26.2KB +128.0B
total +1.0KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
cases 112.1KB 112.7KB +600.0B
timelines 272.6KB 273.6KB +1.0KB
total +1.6KB
Unknown metric groups

API count

id before after diff
actions 195 196 +1

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

cc @academo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Cases] Use the isDeprecated field to check for deprecated connectors.