Skip to content

Conversation

eedugon
Copy link
Contributor

@eedugon eedugon commented Jul 2, 2025

Adds https://www.elastic.co/guide/en/cloud-enterprise/current/ece-apm-settings.html to the appropriate doc as a mapped_page.

This is part of the work for https://github.com/elastic/docs-projects/issues/494

In a parallel activity we will request the redirect of that page to https://www.elastic.co/docs/deploy-manage/deploy/cloud-enterprise/edit-stack-settings-apm

@eedugon eedugon requested a review from a team as a code owner July 2, 2025 18:29
Copy link

github-actions bot commented Jul 2, 2025

🔍 Preview links for changed docs

Copy link
Collaborator

@shainaraskas shainaraskas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you explain a little more about this? the mapped page has settings on it that are not present here. I know you mentioned there as some weirdness around settings that are no longer valid - is that the case here?

@eedugon
Copy link
Contributor Author

eedugon commented Jul 2, 2025

The settings that the mapped page has are already included in the reference documentation and properly linked within this document.
That's why in this specific case the fix is quite easy (in other cases I need to adapt a bit the narrative).

This is where the settings are being linked:

image

I wouldn't add the settings as they would be duplicate and maintaining them would be a mess.

@eedugon eedugon requested a review from shainaraskas July 3, 2025 16:04
Copy link
Collaborator

@shainaraskas shainaraskas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

poor reading comprehension on my part. thanks edu!

@eedugon
Copy link
Contributor Author

eedugon commented Jul 3, 2025

poor reading comprehension on my part.

not at all, I was very glad you asked the question. I realized I hadn't explained why I didn't really import any of the content of the not-redirected page.

Thanks for being on top of it!!

@eedugon eedugon merged commit 4180767 into main Jul 3, 2025
6 checks passed
@eedugon eedugon deleted the ece_redirect_apm_settings branch July 3, 2025 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants