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

Autosuggest: Major rewrite of adjust_fuzzy_fields #2347

Merged
merged 6 commits into from
Sep 21, 2021
Merged

Conversation

felipeelia
Copy link
Member

Description of the Change

This PR addresses the problem outlined in #2346 and improves the readability of the Autosuggest::adjust_fuzzy_fields() method.

Benefits

  • Some early returns and more comments will (hopefully) make the method easier to understand
  • term_suggest is now used for autosuggest searches

Applicable Issues

Closes #2346

Changelog Entry

Fixed: In addition to post titles, now autosuggest also partially matches taxonomy terms.

@felipeelia felipeelia added this to the 3.6.3 milestone Sep 14, 2021
@felipeelia felipeelia self-assigned this Sep 14, 2021
@felipeelia felipeelia requested a review from Rahmon September 14, 2021 14:17
@felipeelia felipeelia assigned Rahmon and unassigned felipeelia Sep 14, 2021
@Rahmon Rahmon assigned felipeelia and Rahmon and unassigned Rahmon and felipeelia Sep 14, 2021
@felipeelia felipeelia merged commit b4d1fdf into develop Sep 21, 2021
@felipeelia felipeelia deleted the fix/issue-2346 branch September 21, 2021 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: Add back term_suggest to search queries
2 participants