-
Notifications
You must be signed in to change notification settings - Fork 164
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
[Inventory] Remove the new services inventory docs #4342
[Inventory] Remove the new services inventory docs #4342
Conversation
This pull request does not have a backport label. Could you fix it @alaudazzi? 🙏
|
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.
LGTM!
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.
It looks like there are links to the page you deleted in two files that need to be updated or removed:
docs/en/serverless/elastic-entity-model.mdx
(2 instances)docs/en/serverless/apm/apm-ui-services.mdx
(1 instance)
Thank you for your review @colleenmcginnis! docs/en/serverless/elastic-entity-model.mdx is being updated in #4340, so broken links are expected. |
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.
🦭
🚀 Built elastic-dot-co-docs-preview-docs successfully!
|
Description
This PR removes the new services inventory documentation as it is superseded by the new Inventory page (#4341) .
Documentation sets edited in this PR
Check all that apply.
docs/en/observability/*
)docs/en/serverless/*
)docs/en/integrations/*
)Related issues
Checklist
Follow-up tasks
Select one.