-
Notifications
You must be signed in to change notification settings - Fork 41
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
Search description instead of post content #28
Comments
Core doesn't provide a lot of ways to customize search results, somewhat intentionally.
It's tempting to create a A different approach would be to use w.org's Sphinx service, or Elasticsearch (via Jetpack). That seems like it'll be more practical, so I'm going to look into that. |
@tellyworth , is there any documentation around why the plugin directory uses a forked version of the |
WordPress/wordpress.org#33 demonstrates how to integrate w/ Jetpack Search. I think we can mostly do the same things here. |
#307 also has some functionality that should be incorporated: the requested locale should be prioritized rather than exclusive. |
This lets us search the meta description field instead of `post_content`. Fixes #28
Normally Core searches
post_content
, but that probably won't make sense for most patterns, since that's storing the HTML and demo content. It'd make more sense to search the meta description field instead.See #11
One way to do this might be to save the description into
post_excerpt
, instead of a meta field. Then it'd be searched for free. We'd still want to disable searchingpost_content
, though.That'd also require changing the two APIs that proxy this API, which would be a pain. We might also want to search by other post meta per #11, so we might need that functionality anyway. We'll probably need to update them anyway, as the design evolves, so maybe those updates can be batched together after more are settled.
The text was updated successfully, but these errors were encountered: