Skip to content

doc: search only returns results from selected version and fix latest version#9033

Merged
joshuakguo merged 2 commits intoaws:mainfrom
joshuakguo:website-search-fix
Mar 27, 2026
Merged

doc: search only returns results from selected version and fix latest version#9033
joshuakguo merged 2 commits intoaws:mainfrom
joshuakguo:website-search-fix

Conversation

@joshuakguo
Copy link
Copy Markdown
Contributor

@joshuakguo joshuakguo commented Mar 24, 2026

Fixes #3172 and #8349

Description

makes the website search only return results of the current version

also makes the search better since we're returning more different entries rather than just the same entry from different versions, for example - the "settings" search from the issue is more helpful now

note: needed to pull in https://github.com/google/docsy/blob/main/assets/js/offline-search.js and edit it but the search.js that's already there is also the same so unsure if additional attributions are needed

How was this change tested?

tested locally when i deployed the website

Does this change impact docs?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 24, 2026

Preview deployment ready!

Preview URL: https://pr-9033.d18coufmbnnaag.amplifyapp.com

Built from commit 32ac91b103ef3f6a5423bc27feeded215dc2fb6d

@joshuakguo joshuakguo marked this pull request as ready for review March 24, 2026 21:37
@joshuakguo joshuakguo requested a review from a team as a code owner March 24, 2026 21:37
@joshuakguo joshuakguo changed the title fix: search only returns results from selected version and fix latest version doc: search only returns results from selected version and fix latest version Mar 24, 2026
@joshuakguo joshuakguo force-pushed the website-search-fix branch 2 times, most recently from 0bcec5e to f90e899 Compare March 27, 2026 21:17
Copy link
Copy Markdown
Contributor

@ryan-mist ryan-mist left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@joshuakguo joshuakguo enabled auto-merge (squash) March 27, 2026 21:26
@joshuakguo joshuakguo merged commit c2bbf02 into aws:main Mar 27, 2026
21 of 22 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.

Docs search should return newer docs versions first

2 participants