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 option to search in specific collections #1369

Merged
merged 4 commits into from
Nov 28, 2017

Conversation

nickgarlis
Copy link
Contributor

@nickgarlis nickgarlis commented Nov 27, 2017

This is an enchancement for search results. With this you can turn on/off search for specific collections with search: true/ search: false. In this example search is disabled for recipies. If you try searching for "chocolate" it will not come up with any results, however if you try searching for "oatmeal"
it will come up with the post in which search is enabled.

@mmistakes
Copy link
Owner

To avoid having to set search: true in every document (or as a front matter default), what if the default behavior is everything is included in the search index unless it has search: false?

Then you can do this instead:

{% unless doc.search == false %}
...
{% endunless %}

@nickgarlis
Copy link
Contributor Author

Good idea, that would make more sense.

@mmistakes mmistakes merged commit 599712e into mmistakes:master Nov 28, 2017
mmistakes added a commit that referenced this pull request Nov 28, 2017
kkunapuli pushed a commit to kkunapuli/kkunapuli.github.io that referenced this pull request May 30, 2019
kkunapuli pushed a commit to kkunapuli/kkunapuli.github.io that referenced this pull request May 30, 2019
sumeetmondal pushed a commit to sumeetmondal/sumeetmondal.github.io that referenced this pull request Sep 10, 2019
sumeetmondal pushed a commit to sumeetmondal/sumeetmondal.github.io that referenced this pull request Sep 10, 2019
jchwenger pushed a commit to jchwenger/jchwenger.github.io that referenced this pull request May 5, 2023
jchwenger pushed a commit to jchwenger/jchwenger.github.io that referenced this pull request May 5, 2023
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.

2 participants