Skip to content

[Streams] Update EditPolicyModal copies#254854

Merged
SoniaSanzV merged 1 commit intoelastic:mainfrom
SoniaSanzV:editPolicyModal/update_copy
Feb 25, 2026
Merged

[Streams] Update EditPolicyModal copies#254854
SoniaSanzV merged 1 commit intoelastic:mainfrom
SoniaSanzV:editPolicyModal/update_copy

Conversation

@SoniaSanzV
Copy link
Contributor

Part of #254852

Summary

From the analysis @mdbirnstiehl did:

For the title, how about "This update affects 6 streams and 4 indices"

For the managed policy warning, how about the following to maybe suggest saving as a new policy:

Modifying a managed policy
This policy is managed by Elasticsearch. Modifying it can cause unexpected behavior. To avoid unintended changes, save as a new policy instead.

For the body text:
7 streams and 5 indices use the ILM policy you're updating. To apply your changes to only this stream, save as a new policy.

For the CTA:

How about [Update current policy][Save as new policy]? Maybe these are too long and might not look quite right, but if they fit, I think this is more explicit.

How to test

View the component in Storybook:

yarn storybook streams_app

@SoniaSanzV SoniaSanzV self-assigned this Feb 25, 2026
@SoniaSanzV SoniaSanzV requested a review from a team as a code owner February 25, 2026 07:49
@SoniaSanzV SoniaSanzV added Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Feature:Streams This is the label for the Streams Project labels Feb 25, 2026
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-management (Team:Kibana Management)

@SoniaSanzV SoniaSanzV changed the title Update EditPolicyModal copies [Streams] Update EditPolicyModal copies Feb 25, 2026
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #11 / ChangePasswordModal render disables submit button when passwords do not match
  • [job] [logs] Jest Tests #11 / IncludeExcludeComponent should prevent identical include and exclude values when creating multiple options

Metrics [docs]

Async chunks

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

id before after diff
streamsApp 1.8MB 1.8MB +83.0B

cc @SoniaSanzV

@SoniaSanzV SoniaSanzV merged commit 7e79080 into elastic:main Feb 25, 2026
27 checks passed
@SoniaSanzV SoniaSanzV deleted the editPolicyModal/update_copy branch February 25, 2026 09:13
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:Streams This is the label for the Streams Project release_note:skip Skip the PR/issue when compiling release notes Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants