Repository navigation
Update Observability integration steps for direct Add integration navigation - #9252
Conversation
Elastic Docs AI PR menuCheck the box to run an AI review for this pull request.
Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team. |
2cb8662 to
b0c0083
Compare
6ebde28 to
886f383
Compare
…igation Co-authored-by: Cursor <cursoragent@cursor.com>
886f383 to
570a92b
Compare
✅ Elastic Docs Style Checker (Vale)No issues found on modified lines! The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
Docs review summaryIssue satisfactionSatisfied. The 16 Observability pages cover the direct Add page navigation, the conditional Ready to add your first integration? step, and the wording alignment that the PR description lists. Review coverage
|
…ce on the AWS pages Co-authored-by: Cursor <cursoragent@cursor.com>
alexandra5000
left a comment
There was a problem hiding this comment.
Nice! I left a couple of comments
Relates to #8153. Part of a set of PRs for this issue, split by docs set:
The PRs don't depend on each other and can merge in any order. Close #8153 after all four merge.
Summary
Two UI changes affect the steps for adding an integration:
How the steps change
applies_tobadge on purpose. Even in the releases that still have it, the page only appears under the conditions above, so the step must stay conditional either way. Readers who don't see the page skip the step, and everyone reaches the same configuration page next. A badge would only add noise. Where this step was a tip or note, it's now a numbered step, because readers who see that page can't continue without it.Changes
16 pages across Cloud, infrastructure and hosts, APM, and logs.
Not changed: the Elastic GovCloud and Firehose pages, because those integrations still open their details page.
Generative AI disclosure
Tool(s) and model(s) used: Cursor with Claude Opus 5.5