Skip to content

Move query languages section under Elasticsearch reference in navigation#2798

Open
Copilot wants to merge 3 commits intomainfrom
copilot/update-navigation-for-query-languages
Open

Move query languages section under Elasticsearch reference in navigation#2798
Copilot wants to merge 3 commits intomainfrom
copilot/update-navigation-for-query-languages

Conversation

Copy link
Contributor

Copilot AI commented Feb 26, 2026

The query languages section was a top-level sibling of the Elasticsearch reference in the nav, but belongs nested as a child of it alongside other Elasticsearch-specific reference content, positioned just after the REST APIs section.

Changes

  • config/navigation.yml: Moved elasticsearch://reference/query-languages from a standalone entry under reference to the first child of elasticsearch://reference/elasticsearch, placing it just after the built-in toc items (which include REST APIs)
# Before: top-level sibling under reference
- toc: elasticsearch://reference/query-languages
  path_prefix: reference/query-languages

# After: first child of elasticsearch reference (just under REST APIs)
- toc: elasticsearch://reference/elasticsearch
  path_prefix: reference/elasticsearch
  children:
    - toc: elasticsearch://reference/query-languages   # ← moved here, first child
      path_prefix: reference/query-languages
    - toc: elasticsearch://reference/text-analysis
    ...

Note: REST APIs is defined within the elasticsearch://reference/elasticsearch toc itself rather than as an explicit child in navigation.yml, so positioning query-languages as the first child places it directly after the built-in toc items (Configuration through Command-line tools, including REST APIs). Page URLs are unchanged — path_prefix: reference/query-languages is preserved — so no redirects are needed.


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

… in navigation.yml

Co-authored-by: leemthompo <32779855+leemthompo@users.noreply.github.com>
Copilot AI changed the title [WIP] Make query languages section a child of Elasticsearch reference Move query languages section under Elasticsearch reference in navigation Feb 26, 2026
@leemthompo
Copy link
Contributor

Copy link
Contributor Author

Copilot AI commented Feb 26, 2026

@copilot do we need to update https://github.com/elastic/elasticsearch/blob/main/docs/reference/toc.yml too?

No, we don't need to update the elasticsearch repo's toc.yml. The toc.yml files in individual repos (like elasticsearch) define the internal content structure of their sections. The config/navigation.yml in docs-builder is the assembly configuration that controls how those sections are nested in the global navigation. Moving elasticsearch://reference/query-languages to be a child of elasticsearch://reference/elasticsearch is purely a docs-builder assembly change.

@leemthompo
Copy link
Contributor

@copilot won't the new URLs change for these pages, meaning we'll need redirects?

Copy link
Contributor Author

Copilot AI commented Feb 26, 2026

@copilot won't the new URLs change for these pages, meaning we'll need redirects?

No redirects needed. The path_prefix for the query-languages section remains reference/query-languages in both the old and new positions — only the navigation hierarchy changes (where it appears in the sidebar), not the actual page URLs.

@leemthompo leemthompo marked this pull request as ready for review February 26, 2026 16:16
@leemthompo leemthompo requested review from a team as code owners February 26, 2026 16:16
@leemthompo leemthompo requested a review from reakaleek February 26, 2026 16:16
@leemthompo
Copy link
Contributor

@georgewallace any preference for where we this to sit in the elasticsearch TOC?

Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

Let's leave this move until after all the releases go out today, to give us some space :)

@leemthompo
Copy link
Contributor

@copilot let's move that section to sit just under REST API

Co-authored-by: leemthompo <32779855+leemthompo@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 27, 2026

@copilot let's move that section to sit just under REST API

Done in b3a91d7. Moved query-languages to be the first item in the elasticsearch reference children list — since REST APIs is defined within the elasticsearch://reference/elasticsearch toc itself (not a separate child entry in navigation.yml), this positions query-languages to appear just after the built-in toc items (Configuration through Command-line tools, which includes REST APIs).

@leemthompo
Copy link
Contributor

leemthompo commented Feb 27, 2026

@leemthompo just realized that there are assembly previews now and so he wasted time doing this locally yesterday 🫥

👏 for that in any case, so NICE 🪄

@elastic/docs-engineering feel free to merge this whenever it makes sense for y'all

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants