Skip to content

Conversation

@enriquegh
Copy link
Contributor

This PR makes two key improvements to our Algolia search implementation:

  1. Increases Maximum Results
  • Added maxResultsPerGroup parameter to Algolia search component
  • Increased limit from default to 20 results per group
  • Improves search experience by showing more relevant matches
  1. Configuration Refactoring
  • Moved Algolia credentials (appId, indexName, apiKey) to centralized searchMetadata config
  • Improves maintainability by consolidating search settings in one location
  • Makes configuration updates more manageable

Testing

Manually after merge

Move hardcoded Algolia credentials (appId, indexName, apiKey) to a centralized searchMetadata configuration object. This change improves maintainability by consolidating search configuration in a dedicated location.
Adds maxResultsPerGroup configuration parameter to Algolia search component
and sets it to 20 results per group in search metadata configuration.
@vercel
Copy link

vercel bot commented Nov 28, 2024

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

Name Status Preview Comments Updated (UTC)
sourcegraph-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 28, 2024 0:22am

Copy link
Contributor

@MaedahBatool MaedahBatool left a comment

Choose a reason for hiding this comment

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

LGTM thank you

@enriquegh enriquegh merged commit 0a62f3e into main Dec 9, 2024
5 checks passed
@enriquegh enriquegh deleted the eg-max-results branch December 9, 2024 19:32
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