Skip to content

Conversation

leemthompo
Copy link
Contributor

@leemthompo leemthompo commented Jul 23, 2025

Contributes to https://github.com/elastic/developer-docs-team/issues/324

It's more intuitive for these examples to live with the main retrievers references

Notes for reviewers

Please signal any example that needs applies_to tags for 9.1, i.e. which were unavailable in 9.0.

@leemthompo leemthompo self-assigned this Jul 23, 2025
@elasticsearchmachine elasticsearchmachine added v9.2.0 needs:triage Requires assignment of a team area label labels Jul 23, 2025
@leemthompo leemthompo added >docs General docs changes auto-backport Automatically create backport pull requests when merged v9.1.0 and removed needs:triage Requires assignment of a team area label v9.2.0 labels Jul 23, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Docs Meta label for docs team label Jul 23, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

Copy link
Contributor

github-actions bot commented Jul 23, 2025

@leemthompo leemthompo changed the title [DOCS] Move retrievers examples page into from docs-content repo [DOCS] Move retrievers examples page from docs-content repo Jul 23, 2025
Copy link
Contributor

@Mikep86 Mikep86 left a comment

Choose a reason for hiding this comment

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

LGTM!



A more specific example of highlighting in RRF can also be found in the [retrievers examples](docs-content://solutions/search/retrievers-examples.md#retrievers-examples-highlighting-retriever-results) page.
A more specific example of highlighting in RRF can also be found in the [retrievers examples](retrievers//retrievers-examples.md#retrievers-examples-highlighting-retriever-results) page.
Copy link
Contributor

Choose a reason for hiding this comment

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

For my learning: Why does this link use the retrievers// prefix, while others use retrievers/?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ha that's a typo, but it seems like CI is lenient on this? TIL.

I'll fix it anyway to avoid confusing people :)

@leemthompo leemthompo merged commit 61a0ddd into elastic:main Jul 24, 2025
10 checks passed
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
9.1

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

Labels

auto-backport Automatically create backport pull requests when merged >docs General docs changes Team:Docs Meta label for docs team v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants