Skip to content

Search: Adding search to ospo-guide#106

Merged
natalialuzuriaga merged 3 commits into
mainfrom
DSACMS/nat/search
Mar 10, 2026
Merged

Search: Adding search to ospo-guide#106
natalialuzuriaga merged 3 commits into
mainfrom
DSACMS/nat/search

Conversation

@natalialuzuriaga

@natalialuzuriaga natalialuzuriaga commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

Search: Adding search to ospo-guide

Problem

ospo-guide is home to 42+ guides so we would like to add search using searchgov to assist users with finding content across our vast collection of guides.

Solution

  • Add searchgov variables to site.yaml for configuration
  • Add search box to frontend

Result

Search box is now included as part of the top toolbar/ menu.

Test Plan

Only works on prod, will assess once this PR is merged!

Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
@natalialuzuriaga natalialuzuriaga self-assigned this Jan 26, 2026
@natalialuzuriaga

Copy link
Copy Markdown
Contributor Author

Notes on this loop:

  • Looks like the API is inactive at the moment according to searchgov dashboard. Thus, it may be best to hold off on merging to prod since search functionality does not work:

After careful consideration and analysis of usage data, SearchGov will be providing web results and text best bets only after February 2026. Please email search@gsa.gov if you have any questions about how this may affect your site's search experience.

  • Would we like results within the page (inline) instead of sending to user to search.gov?

Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
@natalialuzuriaga natalialuzuriaga marked this pull request as ready for review March 9, 2026 22:41

@decause-gov decause-gov 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.

LGTM +1 🚢 🔍

Comment thread _includes/menu.html
@natalialuzuriaga natalialuzuriaga merged commit 19ef451 into main Mar 10, 2026
5 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.

2 participants