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

Allow some pages to be excluded from search index #269

Closed
damithc opened this issue Jun 20, 2018 · 6 comments · Fixed by #296
Closed

Allow some pages to be excluded from search index #269

damithc opened this issue Jun 20, 2018 · 6 comments · Fixed by #296

Comments

@damithc
Copy link
Contributor

damithc commented Jun 20, 2018

Context: In some websites, there is heavy duplication of content. e.g., normal text book and a printable version of a text book.

Suggestion: Allow some pages to be excluded from search results

Benefit: Author has a way to omit duplicated content from search results

Notes:
Allow a searchable: no (or something similar) in frontmatter?

@damithc
Copy link
Contributor Author

damithc commented Jun 20, 2018

@nusjzx can give this one a try? I would like to have this feature before next semester starts.

@yamgent
Copy link
Member

yamgent commented Jun 20, 2018

Allow a searchable: no (or something similar) in frontmatter?

Would it help if we support this in site.json as well? Using globs to specify an entire directory for exclusion could be useful.

@damithc
Copy link
Contributor Author

damithc commented Jun 20, 2018

Would it help if we support this in site.json as well? Using globs to specify an entire directory for exclusion could be useful.

Yes, in fact it is the better option. That way, a parent site can exclude pages from a child site, without having to modify the page source itself.

@nusjzx
Copy link
Contributor

nusjzx commented Jun 29, 2018

@damithc Do we still allow it in frontmatter, or site.json only

@damithc
Copy link
Contributor Author

damithc commented Jun 29, 2018

site.json is enough?

@nusjzx
Copy link
Contributor

nusjzx commented Jun 29, 2018

ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants