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

Limit courses in search results to only show 'featured' courses #286

Closed
hlashbrooke opened this issue Nov 25, 2021 · 0 comments · Fixed by #287
Closed

Limit courses in search results to only show 'featured' courses #286

hlashbrooke opened this issue Nov 25, 2021 · 0 comments · Fixed by #287
Labels
[Component] Learn Theme Website development issues related to the Learn theme. Priority - High High priority issue. [Type] Enhancement New feature request for the Learn website.

Comments

@hlashbrooke
Copy link
Collaborator

We have a few courses on Learn that are essentially placeholders and containers for content that we do not want to be linked anywhere on the site. They don't display on the homepage or on the courses archive page, but they do show up in search results.

We need to limit posts from course post type to only show if the meta field _course_featured has a value of featured. This must not affect other post types, even though they all show up in the search results.

@hlashbrooke hlashbrooke added [Component] Learn Theme Website development issues related to the Learn theme. Priority - High High priority issue. [Type] Enhancement New feature request for the Learn website. labels Nov 25, 2021
hlashbrooke added a commit that referenced this issue Nov 25, 2021
This PR removes non-featured courses from search results without affecting other post types. It also removes pages from search results.

Fixes #286
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Component] Learn Theme Website development issues related to the Learn theme. Priority - High High priority issue. [Type] Enhancement New feature request for the Learn website.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant