Skip to content
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

Replace APM integration documentation with static page #2008

Closed
bmorelli25 opened this issue Jul 20, 2022 · 4 comments · Fixed by elastic/apm-server#9932
Closed

Replace APM integration documentation with static page #2008

bmorelli25 opened this issue Jul 20, 2022 · 4 comments · Fixed by elastic/apm-server#9932
Assignees
Labels
Team:Docs Label for the Observability docs team v8.6.0

Comments

@bmorelli25
Copy link
Member

bmorelli25 commented Jul 20, 2022

Summary

Remove documentation from the APM integration so we don't have to update docs in two places.

@bmorelli25 bmorelli25 added the Team:Docs Label for the Observability docs team label Jul 20, 2022
@bmorelli25 bmorelli25 self-assigned this Jul 20, 2022
@bmorelli25
Copy link
Member Author

@axw — is this issue still relevant? I read through elastic/kibana#137759, but I'm struggling to determine how the final decision impacts this issue.

@axw
Copy link
Member

axw commented Oct 10, 2022

@bmorelli25 it's not necessary as we are still publishing packages. IMO it would still be a good thing to do to avoid fragmentation/skew across the main docs and the integration docs. @simitt WDYT?

@simitt
Copy link
Contributor

simitt commented Oct 10, 2022

A static page that links to the APM guide of the according version sounds good to me. Having to maintain multiple document sources for the APM integration is error prone.

@ghost
Copy link

ghost commented Jan 13, 2023

Hi @bmorelli25

As per the ticket https://github.com/elastic/apm-server/pull/9932/files we have validated changes and found updated changes are available.

APM changes links:

Screenshot:
image

Please let us know if any else validation is required.
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Docs Label for the Observability docs team v8.6.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants