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 related_posts option RELATED_POSTS_SKIP_SAME_CATEGORY #380

Merged
merged 1 commit into from Dec 30, 2014

Conversation

Projects
None yet
3 participants
@kernc
Copy link
Contributor

commented Dec 23, 2014

Posts in the same category may sometimes (depending on the overall site design) have a lot of tags in common. Thus the article.related_posts holds only the posts that are tightly (perhaps too tightly) related to the original post, and no posts from other categories, since those, with only one or two tags in common, can't as effectively compete.

Add related_posts option RELATED_POSTS_SKIP_SAME_CATEGORY
Posts in the same category may sometimes (depending on the
overall site design) have a lot of tags in common. Thus the
`article.related_posts` holds only the posts that are tightly
(perhaps too tightly) related to the original post, and no
posts from other categories, since those, with only one or two
tags in common, can't as effectively compete.
@kernc

This comment has been minimized.

Copy link
Contributor Author

commented Dec 24, 2014

If either @avaris or @Shaked are the original contributors, I think they may wish to give their seal of approval.

@avaris

This comment has been minimized.

Copy link
Member

commented Dec 24, 2014

Well, I'm not the original author, but I refactored it to some extend. So, as far as I'm concerned, this looks good.

@kernc

This comment has been minimized.

Copy link
Contributor Author

commented Dec 24, 2014

Indeed, thanks, the original author of the plugin seems to be @samrat.

@justinmayer

This comment has been minimized.

Copy link
Member

commented Dec 30, 2014

Thanks for the enhancement, @kernc.

justinmayer added a commit that referenced this pull request Dec 30, 2014

Merge pull request #380 from kernc/related_posts_skipcategory
Add related_posts option RELATED_POSTS_SKIP_SAME_CATEGORY

@justinmayer justinmayer merged commit 242f221 into getpelican:master Dec 30, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.