-
Notifications
You must be signed in to change notification settings - Fork 166
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
Clean up APM getting started intro page #4599
Conversation
A documentation preview will be available soon. Request a new doc build by commenting
If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here. |
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.
I'm good with these changes. The Get Started page is now a quick overview, a brief pro/con list of each option, and the flowchart to help people decide. I think there's enough context on the page for users to decide which option is easiest for their use case.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
@colleenmcginnis , totally! sorry for having jumped in without a clear scope of the changes! 🙏 I'll create an issue for the APM installation guide and its relation with the Fleet Setup, which I think it's probably the most important part of my comment. |
* clean up apm getting started intro page * Update docs/en/observability/apm/getting-started-apm/index.asciidoc (cherry picked from commit 494830b)
* clean up apm getting started intro page * Update docs/en/observability/apm/getting-started-apm/index.asciidoc (cherry picked from commit 494830b)
* clean up apm getting started intro page * Update docs/en/observability/apm/getting-started-apm/index.asciidoc (cherry picked from commit 494830b)
* clean up apm getting started intro page * Update docs/en/observability/apm/getting-started-apm/index.asciidoc (cherry picked from commit 494830b) Co-authored-by: Colleen McGinnis <[email protected]>
* clean up apm getting started intro page * Update docs/en/observability/apm/getting-started-apm/index.asciidoc (cherry picked from commit 494830b) Co-authored-by: Colleen McGinnis <[email protected]>
* clean up apm getting started intro page * Update docs/en/observability/apm/getting-started-apm/index.asciidoc (cherry picked from commit 494830b) Co-authored-by: Colleen McGinnis <[email protected]>
Description
Cleans up the APM getting started intro page as discussed in #4336 including:
Documentation sets edited in this PR
Check all that apply.
docs/en/observability/*
)docs/en/serverless/*
)docs/en/integrations/*
)Related issue
Closes #4336
Checklist
Follow-up tasks
Select one.