Skip to content

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Aug 20, 2025

This PR must be merged after elastic/elasticsearch#133362

It removes the query DSL API quickstart from docs-content since the other PR moves it to the Elasticsearch repo and a new section of the documentation.

Copy link

github-actions bot commented Aug 20, 2025

@leemthompo
Copy link
Contributor

Hey @lcawl I would be adamant that we not put this in Explore & Analyze, as you mentioned we've discussed this at lot with Florent and like the ES|QL section, the bulk of the rest of the query language stuff should also eventually be re-homed back to the elasticsearch repo. Stuff is too scattered today.

So I'd say either move this somewhere else in the solutions/use case or into the ES repo :)

@lcawl
Copy link
Contributor Author

lcawl commented Aug 21, 2025

So I'd say either move this somewhere else in the solutions/use case or into the ES repo :)

Yeah, I found #1808 and elastic/elasticsearch#133301 only after doing this quick draft so I'll follow the pattern of we're moving to with ES|QL and close this PR.

@lcawl lcawl marked this pull request as ready for review August 22, 2025 21:18
@lcawl lcawl requested review from a team as code owners August 22, 2025 21:18
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.

LGTM, thank you!

@leemthompo
Copy link
Contributor

Setting this to merge to minimize risks of SEO duplication issues :)

@leemthompo leemthompo enabled auto-merge (squash) August 25, 2025 10:00
@leemthompo leemthompo merged commit b8e2818 into elastic:main Aug 25, 2025
6 of 7 checks passed
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