Ensures related_posts are only set for a post #4620

Merged
merged 1 commit into from Mar 2, 2016

Conversation

Projects
None yet
3 participants
@jeffkole
Contributor

jeffkole commented Mar 1, 2016

Prior to this change, the related posts for the most recently rendered post
stayed set on the site object. This could result in pages that showed related
posts even when the page represented an entire collection of posts, such as on
an index page. This change restores the functionality from Jekyll V2.

Ensures related_posts are only set for a post
Prior to this change, the related posts for the most recently rendered post
stayed set on the `site` object. This could result in pages that showed related
posts even when the page represented an entire collection of posts, such as on
an index page.  This change restores the functionality from Jekyll V2.
@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Mar 1, 2016

Member

I agree with this! Thank you for sending the patch with a test. 🎉

Once Travis is done doing its thing, I'll merge if all looks good. 👍

Member

parkr commented Mar 1, 2016

I agree with this! Thank you for sending the patch with a test. 🎉

Once Travis is done doing its thing, I'll merge if all looks good. 👍

@parkr parkr added the fix label Mar 1, 2016

@parkr parkr added this to the 3.2 milestone Mar 1, 2016

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Mar 2, 2016

Member

@jekyllbot: merge +bug

Member

parkr commented Mar 2, 2016

@jekyllbot: merge +bug

jekyllbot added a commit that referenced this pull request Mar 2, 2016

@jekyllbot jekyllbot merged commit 11d2d78 into jekyll:master Mar 2, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

jekyllbot added a commit that referenced this pull request Mar 2, 2016

@jeffkole jeffkole deleted the jeffkole:fix/clear-related-posts-for-non-post-pages branch Mar 24, 2016

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