Skip to content

Remove Issue Correlation page#37718

Merged
Vomain merged 3 commits into
masterfrom
thibaultk/remove-issue-correlation-page
Jun 24, 2026
Merged

Remove Issue Correlation page#37718
Vomain merged 3 commits into
masterfrom
thibaultk/remove-issue-correlation-page

Conversation

@Vomain

@Vomain Vomain commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

Removes the Issue Correlation page (/error_tracking/issue_correlation/) and its navigation entries across all language menu files, as the feature no longer exists.

Merge instructions

Merge readiness:

  • Ready for merge

Additional notes

Feature no longer exists; removed the page and all menu entries across EN, FR, ES, JA, and KO nav files.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@Vomain
Vomain requested a review from a team as a code owner June 24, 2026 11:07
@github-actions github-actions Bot added the Architecture Everything related to the Doc backend label Jun 24, 2026
@Vomain
Vomain requested a review from Copilot June 24, 2026 11:07
@github-actions

github-actions Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Removed or renamed files (these should redirect)

Modified Files

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Removes the “Issue Correlation” Error Tracking documentation page and removes its navigation entries from the main menu across supported locales, aligning docs navigation with a feature that no longer exists.

Changes:

  • Deleted content/en/error_tracking/issue_correlation.md.
  • Removed the “Issue Correlation” menu item from config/_default/menus/main.*.yaml (en/es/fr/ja/ko).

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
content/en/error_tracking/issue_correlation.md Removes the English Issue Correlation documentation page entirely.
config/_default/menus/main.en.yaml Drops “Issue Correlation” from the English Error Tracking nav.
config/_default/menus/main.es.yaml Drops “Issue Correlation” from the Spanish Error Tracking nav.
config/_default/menus/main.fr.yaml Drops “Issue Correlation” from the French Error Tracking nav.
config/_default/menus/main.ja.yaml Drops “Issue Correlation” from the Japanese Error Tracking nav.
config/_default/menus/main.ko.yaml Drops “Issue Correlation” from the Korean Error Tracking nav.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread config/_default/menus/main.en.yaml
Comment thread config/_default/menus/main.es.yaml
- Remove content/es/error_tracking/issue_correlation.md (translated page still present)
- Add aliases redirect on _index.md so old URLs don't 404

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.

@Vomain Vomain self-assigned this Jun 24, 2026

@maycmlee maycmlee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! Updates to translation docs need to go through the translation pipeline, so you might see those updates get reverted. Feel free to merge if this is ready.

@Vomain
Vomain merged commit e751cc9 into master Jun 24, 2026
28 of 29 checks passed
@Vomain
Vomain deleted the thibaultk/remove-issue-correlation-page branch June 24, 2026 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Architecture Everything related to the Doc backend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants