Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add search functionality and upgrade docusaurus #89

Merged
merged 15 commits into from
Mar 14, 2024
Merged

Conversation

john-rock
Copy link

@john-rock john-rock commented Mar 13, 2024

Asana Task

Description

This PR inlcudes the following work required to add a search feature on handbook.getdbt.com

image

Additional Notes

With the docusaurus v2 upgrade, you'll notice slight UI changes. Creating new and editing existing items should be done the same by modifying the files within the /docs directory.
Talent_Development___Handbook

After testing the local search options noted on the docusaurs page here I've determined the plugin in this PR will be the best option for us.

The search plugin only works on a production build of the site. If you need to test locally you'll need to run the command npm run build and then npm run serve.

This plugin is a local search so we have much less control than we would using a hosted search index, such as Algolia.

Preview

https://handbook-getdbt-com-git-feature-add-search-dbt-labs.vercel.app

Loom Walkthrough

https://www.loom.com/share/6937f93f6e4f40fe97cd7fa9e9f6fabf

Copy link

vercel bot commented Mar 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
handbook-getdbt-com ✅ Ready (Inspect) Visit Preview Mar 13, 2024 5:56pm

@john-rock john-rock changed the title Feature/add search Add search functionality and upgrade docusaurus Mar 13, 2024
Copy link

@leighstaub leighstaub left a comment

Choose a reason for hiding this comment

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

Looks great, thanks @john-rock ! Excited to have this feature added-- thanks for making it happen.

Copy link

@JKarlavige JKarlavige left a comment

Choose a reason for hiding this comment

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

lgtm, search features looks like it works great! :shipit:

@john-rock john-rock merged commit 2d9b097 into main Mar 14, 2024
4 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.

3 participants