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

feat: Use Algolia Docsearch #163

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

Conversation

IgnaceMaes
Copy link
Member

@IgnaceMaes IgnaceMaes commented Aug 5, 2023

Switches from the current custom built components to the premade Docsearch components by Algolia.

The biggest benefit would be accessibility: these components are fully keyboard accessible.

ref:
https://github.com/algolia/docsearch

@netlify
Copy link

netlify bot commented Aug 5, 2023

Deploy Preview for guidemaker-ember-template ready!

Name Link
🔨 Latest commit 70647d6
🔍 Latest deploy log https://app.netlify.com/sites/guidemaker-ember-template/deploys/64cf468b3dfd4c00084f99a3
😎 Deploy Preview https://deploy-preview-163--guidemaker-ember-template.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.


export default Component.extend({
layout,

classNames: ['search-input'],

searchService: service('search'),
Copy link
Member Author

Choose a reason for hiding this comment

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

This service can likely also be removed, but unclear if that would be a breaking change for consumers of this package.

@IgnaceMaes
Copy link
Member Author

Tested this locally on guides-source and while the search queries do give results, they aren't showing up in the UI.

I wonder if this is a backend thing that's not compatible with the indexes?

Should this be registered separately? https://docsearch.algolia.com/apply/

@IgnaceMaes IgnaceMaes marked this pull request as draft April 19, 2024 12:54
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.

1 participant