Skip to content

Conversation

marciw
Copy link
Contributor

@marciw marciw commented Mar 13, 2025

Double-check raw migrated files used in the Troubleshoot section and delete the ones we don't need anymore.

@shainaraskas one fyi for you: I confirmed that ece-find-md
is now represented in both Troubleshoot and Deploy & Manage, according to our divine mappings 🔮

@marciw marciw marked this pull request as ready for review March 13, 2025 20:35
@eedugon
Copy link
Contributor

eedugon commented Mar 14, 2025

thanks @marciw !

@shainaraskas , I see the page is mapped in both places:

  • deploy-manage/deploy/cloud-enterprise/search-filter-deployments.md
  • troubleshoot/deployments/cloud-enterprise/cloud-enterprise.md

In my opinion the page belongs to deploy and manage, as it's about filtering deployments (can be useful in troubleshooting to filter unhealthy deployments) but it's not directly related with troubleshooting at all. The title is misleading so we can also update the title and remove the finding problems from it.

Where should we fix this? If we agree I can create a quick PR and remove the duplicated mapping.

@marciw
Copy link
Contributor Author

marciw commented Mar 14, 2025

thanks @marciw !

@shainaraskas , I see the page is mapped in both places:

  • deploy-manage/deploy/cloud-enterprise/search-filter-deployments.md
  • troubleshoot/deployments/cloud-enterprise/cloud-enterprise.md

In my opinion the page belongs to deploy and manage, as it's about filtering deployments (can be useful in troubleshooting to filter unhealthy deployments) but it's not directly related with troubleshooting at all. The title is misleading so we can also update the title and remove the finding problems from it.

Where should we fix this? If we agree I can create a quick PR and remove the duplicated mapping.

Shaina and I already discussed this during the mapping phase, and it's in the spreadsheet with both mappings... in the troubleshooting section, the page is called "Finding deployments" and we have plans to tailor it.

Happy for you to do whatever you like in D&M though of course :)

@marciw marciw merged commit 2cf3461 into main Mar 14, 2025
4 checks passed
@marciw marciw deleted the mw-raw-migrated-cleanup branch March 14, 2025 13:29
@shainaraskas
Copy link
Collaborator

we need to remove the duplicated mapped_pages value if it exists (I'd probably prefer linking to the deploy and manage one) but I'm ok with the content being duplicated (prob need to use a snippet if we don't see it diverging soon)

@marciw
Copy link
Contributor Author

marciw commented Mar 14, 2025

we need to remove the duplicated mapped_pages value if it exists (I'd probably prefer linking to the deploy and manage one) but I'm ok with the content being duplicated (prob need to use a snippet if we don't see it diverging soon)

gotcha.

@eedugon the mapped_pages value is correct in the D&M file

#784 removes the mapping from the troubleshooting file

@marciw marciw mentioned this pull request Mar 14, 2025
marciw added a commit that referenced this pull request Mar 14, 2025
Prefer [the D&M
page](https://github.com/elastic/docs-content/blob/main/deploy-manage/deploy/cloud-enterprise/search-filter-deployments.md)
rather than the troubleshooting page. We have intentionally duplicated
content in the troubleshooting section (for now).

see
#778 (comment)
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.

4 participants