-
Notifications
You must be signed in to change notification settings - Fork 89
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
[components]: Search functionality feature on website #78
Conversation
❌ Deploy Preview for spectacular-dragon-c1015c failed.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks fine to me - one thing missing are some short docs. Docs should probably be split into:
- user section explaining what search is and maybe short screenshot.
- admin section explaining how to configure (e.g. what fields need to be set where)
@khalilcodes this now has conflicts so i can't merge. |
…34-feature-search
Yes will merge along with added docs. |
ea9b8ce
to
2267c65
Compare
The build seems to be failing here for some reason (suspecting package-lock after merge). Here's some more info on why the build can fail - https://nextjs.org/docs/messages/failed-loading-swc I will close this and create a new branch with PR as the commits have become messy here. |
Search component
Tasks from #34
This PR adds a doc search feature to the website.
Tasks
Notes
Required to add algolia configs as env variables:
Screenshots