Skip to content

Conversation

@mich-elle-luna
Copy link
Collaborator

  • Add data-summary attribute to integration articles in list template
  • Update integrations-filters.js to search both title and summary text
  • Now searches for keywords like 'redis-py' and 'lettuce' in descriptions
  • Previously only searched in titles/headings, missing important keywords

This fixes the issue where searching for client library names like 'redis-py' or 'lettuce' returned no results even though they were mentioned in the integration descriptions.

- Add data-summary attribute to integration articles in list template
- Update integrations-filters.js to search both title and summary text
- Now searches for keywords like 'redis-py' and 'lettuce' in descriptions
- Previously only searched in titles/headings, missing important keywords

This fixes the issue where searching for client library names like
'redis-py' or 'lettuce' returned no results even though they were
mentioned in the integration descriptions.
@github-actions
Copy link
Contributor

github-actions bot commented Oct 15, 2025

DOC-5826

@mich-elle-luna mich-elle-luna requested a review from a team October 15, 2025 23:33
Copy link
Contributor

@andy-stark-redis andy-stark-redis left a comment

Choose a reason for hiding this comment

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

Big improvement :-) The only slight issues I could find were:

  • If you type "c" or "c", it doesn't bring "C client for Redis" to the top of the list (this is admittedly a strange example that hardly anyone will look for). C# is still OK, btw.
  • Is "Redis" a stopword? If you type "red" or "redis", it seems like it should bring "RedisVL", "RedisOM", "Redis on Railway", etc, to the top of the list, but it doesn't seem to do that.
  • Typing "ri" only brings RIOT to the second position.
  • Maybe we should add a parenthesised "RDI" to the RDI tile's title? It's not the first three letters, but it's an obvious thing for people to type. Fingers crossed that we don't get a new product called "Ferdinand" or something :-)

These are minor things, though. I'd certainly accept this as it is, so I will approve.

@mich-elle-luna mich-elle-luna merged commit 223ad03 into main Oct 16, 2025
8 checks passed
@mich-elle-luna mich-elle-luna deleted the DOC-5826 branch October 16, 2025 18:11
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