-
Notifications
You must be signed in to change notification settings - Fork 159
Add reference page for default log component templates #2689
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
Conversation
🔍 Preview links for changed docs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work, and thanks for putting this together! I've added a few suggestions, let me know if you have any questions or concerns, and I'll be happy to discuss.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this! Added some minor comments
@LucaWintergerst can you please have a look? The last part encourages users to follow the |
@gbamparop let's make note of this somewhere and follow up, but I would not change anything at this time - we'll first need to ship 9.2 and we'll also need more coherent docs for Streams at this time with updates to the new UX changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good with the updates! Thanks again!
cc @mdbirnstiehl so we can capture it for the future |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
I'll make a note, I assume we will have quite a bit of Streams docs planning coming up for 9.2. |
This PR adds a new reference page under
/solutions/observability/logs/
that documents the default component templates applied to logs data streams.Closes #1863