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

Smarter and faster plugin and themes searching #169

Merged
merged 9 commits into from
Feb 21, 2025
Merged

Conversation

chuckadams
Copy link
Contributor

@chuckadams chuckadams commented Feb 21, 2025

Pull Request

What changed?

  • Made use of pg_trgm similarity searching on several of the plugin and theme columns.
  • Use UNION ALL query to build search results in order of specificity (slugs, name, short description, then description).

Why did it change?

Our search functionality doesn't have to be as good as .org, but this gets us to acceptable.

Did you fix any specific issues?

None (the search performance issue was closed previously, this is about making it smarter)

CERTIFICATION

By opening this pull request, I do agree to abide by the Code of Conduct and be bound by the terms of the Contribution Guidelines in effect on the date and time of my contribution as proven by the revision information in GitHub.

@chuckadams chuckadams requested review from namithj, costdev and a team February 21, 2025 20:48
@chuckadams chuckadams marked this pull request as ready for review February 21, 2025 20:48
@chuckadams chuckadams merged commit 165fbe3 into main Feb 21, 2025
1 check passed
@chuckadams chuckadams deleted the chuck/dev/search branch February 21, 2025 21:20
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.

2 participants