Skip to content

docs: added Algolia search to website #4187

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

Merged
merged 6 commits into from
Nov 23, 2021

Conversation

JoshuaKGoldberg
Copy link
Member

@JoshuaKGoldberg JoshuaKGoldberg commented Nov 19, 2021

PR Checklist

Overview

Follows https://docusaurus.io/docs/search#using-algolia-docsearch to get Algolia search on the site. 🔍

In my usage this doesn't change the website right now -no header search bar or /search page-, so keeping in draft... I don't know why it's not showing locally. The steps in facebook/docusaurus#4975, run in both the repo root and the packages/website directory aren't fixing it, and I don't see any obvious typos or missing packages. ✔️

@typescript-eslint
Copy link
Contributor

Thanks for the PR, @JoshuaKGoldberg!

typescript-eslint is a 100% community driven project, and we are incredibly grateful that you are contributing to that community.

The core maintainers work on this in their personal time, so please understand that it may not be possible for them to review your work immediately.

Thanks again!


🙏 Please, if you or your company is finding typescript-eslint valuable, help us sustain the project by sponsoring it transparently on https://opencollective.com/typescript-eslint. As a thank you, your profile/company logo will be added to our main README which receives thousands of unique visitors per day.

@netlify
Copy link

netlify bot commented Nov 19, 2021

✔️ Deploy Preview for typescript-eslint ready!

🔨 Explore the source changes: 4d59105

🔍 Inspect the deploy log: https://app.netlify.com/sites/typescript-eslint/deploys/619a766a6689420008db0eb0

😎 Browse the preview: https://deploy-preview-4187--typescript-eslint.netlify.app

@JoshuaKGoldberg JoshuaKGoldberg marked this pull request as ready for review November 19, 2021 19:29
@bradzacher bradzacher changed the title feat: added Algolia search to website docs: added Algolia search to website Nov 19, 2021
@@ -32,6 +32,7 @@ const config = {
customCss: require.resolve('./src/css/custom.css'),
},
],
'@docusaurus/theme-search-algolia',
Copy link
Member

Choose a reason for hiding this comment

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

Just a kind notice: we may introduce shorthands in the next (few) release (facebook/docusaurus#5930) like ESLint's eslint-plugin-react -> plugins: ['react']. In that case you may want to put this in themes instead so you can shorten it to themes: ['search-algolia']

@armano2 armano2 added the documentation Documentation ("docs") that needs adding/updating label Nov 20, 2021
Copy link
Member

@JamesHenry JamesHenry left a comment

Choose a reason for hiding this comment

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

Sorry if I'm missing something but I don't see any search functionality on the latest preview build on this PR?

@@ -16,7 +16,8 @@ interface FeatureItem {

const features: FeatureItem[] = [
{
title: 'What are ESLint and TypeScript, and how do they compare?',
title:
'Hello world: What are ESLint and TypeScript, and how do they compare?',
Copy link
Member Author

Choose a reason for hiding this comment

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

@JamesHenry very odd, I saw the search on my local deployment from the commit in the PR:

Screenshot of local deployment with search

Now, with this Hello World: change, I see it on the Netlify deployment too:

Screenshot of Netlify deployment with Hello World and search

🤷

Copy link
Member

Choose a reason for hiding this comment

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

Might be related to caching... I can't pinpoint it either

@bradzacher
Copy link
Member

Just checked the preview deployment and it's all working for me!
Was able to search up a rule and navigate to it.

Great work!

@bradzacher bradzacher merged commit 46a9aee into typescript-eslint:main Nov 23, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 24, 2021
@JoshuaKGoldberg JoshuaKGoldberg deleted the algolia-search branch March 1, 2022 18:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Documentation ("docs") that needs adding/updating
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Add Algolia search to the website
5 participants