Skip to content

Conversation

kilfoyle
Copy link
Contributor

@kilfoyle kilfoyle commented Aug 20, 2025

The current Update existing data stream page shows API steps but not the equivalent procedure for doing things in the UI, so this adds in those steps.

@yetanothertw This isn't ready for review yet, but regarding our discussion here, I'm thinking that this might be an example of how we can improve the "Data stream lifecycle section" to try and achieve parity with what we now have in the ILM section, if you think that's a good approach.

Please see: preview page
Closes: https://github.com/elastic/docs-content-internal/issues/232

@kilfoyle kilfoyle requested a review from yetanothertw August 20, 2025 21:51
Copy link

github-actions bot commented Aug 20, 2025

🔍 Preview links for changed docs

@kilfoyle kilfoyle marked this pull request as ready for review August 21, 2025 14:50
@kilfoyle kilfoyle requested a review from a team as a code owner August 21, 2025 14:50
Copy link
Contributor

@yetanothertw yetanothertw left a comment

Choose a reason for hiding this comment

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

This looks great, @kilfoyle!

I've left a few comments, but they're just nitpicks. So take them or leave them, I won't mind either way 😄

@kilfoyle kilfoyle enabled auto-merge (squash) August 22, 2025 13:52
@kilfoyle
Copy link
Contributor Author

Thanks for the excellent review @yetanothertw!
I really appreciate having had multiple suggestions to choose from. :-)

@kilfoyle kilfoyle merged commit 1affd66 into elastic:main Aug 22, 2025
7 checks passed
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.

2 participants