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

Add search_keywords to Community model (TS-2350) #1111

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Roffenlund
Copy link
Contributor

  • Add a ArrayField to the Community model to enable slitghtly more optimized search. The field stores an array of strings and matches the search query to these keywords, alongside filtering by community name.

  • This allows users to find communities with short abbreviations or alternative stylings of community names.

  • Add the search_keywords field to CommunityListAPIView search_fields.

  • Implement tests with various test cases for CommunityListAPIView endpoint.

Add a ArrayField to the Community model to enable slitghtly more
optimized search. The field stores an array of strings and matches the
search query to these keywords, alongside filtering by community name.

This allows users to find communities with short abbreviations or
alternative stylings of community names.

Add the search_keywords field to CommunityListAPIView search_fields.

Implement tests with various test cases for CommunityListAPIView
endpoint.

Refs. TS-2350
@Roffenlund
Copy link
Contributor Author

Screenshot 2025-03-05 at 13 02 28

Copy link
Contributor

@x753 x753 left a comment

Choose a reason for hiding this comment

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

Looks good, but perhaps this PR should also include a small change to sync.py so the new field can be populated from the ecosystem schema.

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.

4 participants