Skip to content

Conversation

colleenmcginnis
Copy link
Contributor

@colleenmcginnis colleenmcginnis commented Feb 7, 2025

Related to https://github.com/elastic/docs-projects/issues/412

I'm just making sure all old URLs that should be mapped are indeed mapped.

@shainaraskas: The deploy and manage pages included here shouldn't change -- they are just getting additional mapped pages in the frontmatter.

@florent-leborgne @szabosteve: There are a few new pages in the explore and analyze section. I'm not sure if you actually need them, but they were included via "All children" in the inventory. If they are duplicates of other content, you can just add the mapped page URLs to the appropriate page.

Let me know if you have any questions!

@colleenmcginnis
Copy link
Contributor Author

Note: I will fix the errors in the morning. 😴

@szabosteve
Copy link
Contributor

@colleenmcginnis Thank you for opening this. The pages included in this PR under Machine learning in Kibana were deliberately deleted, as they are virtually duplicates of other pages in the ML guide but with slightly different wording. After I saw the generated content, I decided to delete them via this PR (URLs are handled as part of the PR). The population analysis page, however, is something that we need. Thank you for adding to the MD files.

@shainaraskas
Copy link
Collaborator

no major concerns here (holding until the pr is ready for review to approve), but cc'ing @kosabogi for visibility because she is working on snapshot/restore

@colleenmcginnis
Copy link
Contributor Author

colleenmcginnis commented Feb 7, 2025

The pages included in this PR under Machine learning in Kibana were deliberately deleted

@szabosteve I had a feeling that was the case. Can you please let me know which files include the duplicate content so we know where to redirect these links?

Old AsciiDoc URL v3 file that URL should redirect to
https://www.elastic.co/guide/en/serverless/current/observability-aiops-analyze-spikes.html /explore-analyze/machine-learning/machine-learning-in-kibana/xpack-ml-aiops.md#log-rate-analysis
https://www.elastic.co/guide/en/serverless/current/observability-aiops-detect-anomalies.html /explore-analyze/machine-learning/machine-learning-in-kibana/xpack-ml-anomalies.html
https://www.elastic.co/guide/en/serverless/current/observability-aiops-detect-change-points.html /explore-analyze/machine-learning/machine-learning-in-kibana/xpack-ml-aiops.md#change-point-detection
https://www.elastic.co/guide/en/serverless/current/observability-aiops-forecast-anomalies.html /explore-analyze/machine-learning/anomaly-detection/ml-ad-forecast.html
https://www.elastic.co/guide/en/serverless/current/observability-aiops-tune-anomaly-detection-job.html /explore-analyze/machine-learning/anomaly-detection/anomaly-how-tos.html

@szabosteve
Copy link
Contributor

@colleenmcginnis I updated the table in your comment above. However, my PR that deleted the pages addressed the redirects hopefully correctly. Please let me know if I can help anyhow. Thank you!

@colleenmcginnis colleenmcginnis marked this pull request as ready for review February 10, 2025 15:35
Copy link
Contributor

@szabosteve szabosteve left a comment

Choose a reason for hiding this comment

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

Thank you!

@colleenmcginnis colleenmcginnis merged commit 7aed2f6 into main Feb 10, 2025
5 checks passed
@colleenmcginnis colleenmcginnis deleted the mapped-pages branch February 10, 2025 16:25
wajihaparvez added a commit that referenced this pull request Feb 18, 2025
Closes [#369](elastic/docs-projects#369) 
Preview
[here](https://docs-v3-preview.elastic.dev/elastic/docs-content/pull/464/manage-data/data-store/index-basics)

### Summary

What I changed:

- Combined content from the three mapped pages
- Altered the heading structure to make sense of the added content and
improve readability i.e.: divided the page into **H2: Index components**
and **H2: Index management** + changed all other headings to H3 and
moved them under the relevant H2
- Drafted a new **Manage component templates** subsection to be
consistent with the UI/screenshots
- Cut down extra descriptive content from subsections to limit this page
to Index _basics_ only, but linked out to relevant pages to compensate.
The additions to the Mapping page are from one of the raw migrated files
that I think made more sense there than on Index basics
- Deleted links to Enriching your data examples from the **Manage enrich
policies** subsection because I don't think that's relevant to this page

I'll delete the code comments after this is reviewed. @colleenmcginnis
@kilfoyle let me know what you think of these decisions. I'm open to all
insight/suggestions 😄
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.

3 participants