Skip to content

[Cases] Show deprecated icon in connectors with isDeprecated true#132237

Merged
cnasikas merged 10 commits intoelastic:mainfrom
academo:fix/deprecated-connector-icon
May 20, 2022
Merged

[Cases] Show deprecated icon in connectors with isDeprecated true#132237
cnasikas merged 10 commits intoelastic:mainfrom
academo:fix/deprecated-connector-icon

Conversation

@academo
Copy link
Copy Markdown
Contributor

@academo academo commented May 16, 2022

Summary

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

Closes: #131033 and #130640

image
image
image

Release note

Show a warning for deprecated preconfigured connectors.

@academo academo self-assigned this May 16, 2022
@academo academo added release_note:fix backport:skip This PR does not require backporting Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// Feature:Cases Cases feature v8.3.0 labels May 16, 2022
@academo academo requested review from cnasikas and peteharverson May 16, 2022 12:20
@academo academo marked this pull request as ready for review May 16, 2022 12:20
@academo academo requested a review from a team as a code owner May 16, 2022 12:20
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/response-ops-cases (Feature:Cases)

Copy link
Copy Markdown
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

Tested and LGTM

@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
cases 432 431 -1

Async chunks

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

id before after diff
cases 299.7KB 299.2KB -502.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
cases 81 82 +1

Total ESLint disabled count

id before after diff
cases 97 98 +1

History

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

backport:skip This PR does not require backporting Feature:Cases Cases feature release_note:enhancement Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v8.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

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

7 participants