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

Latest post first in related posts #1271

Merged
merged 1 commit into from Jul 8, 2013
Merged

Conversation

@sanandnarayan
Copy link
Contributor

@sanandnarayan sanandnarayan commented Jul 6, 2013

Fixes issue #1240

@parkr
Copy link
Member

@parkr parkr commented Jul 6, 2013

Nice, thanks! This LGTM. @mattr-, care to take a look?

@ghost ghost assigned mattr- Jul 7, 2013
@mattr-
Copy link
Member

@mattr- mattr- commented Jul 8, 2013

👍 from me.

parkr added a commit that referenced this pull request Jul 8, 2013
Latest post  first in related posts
@parkr parkr merged commit e8d82fd into jekyll:master Jul 8, 2013
1 check passed
1 check passed
default The Travis CI build passed
Details
parkr added a commit that referenced this pull request Jul 8, 2013
@benbalter
Copy link
Contributor

@benbalter benbalter commented Jul 8, 2013

Related posts sans LSI is just latest posts? Has that always been the case? Mind blown.

@mattr-
Copy link
Member

@mattr- mattr- commented Jul 8, 2013

Yup, that's always been the case.

On Mon, Jul 8, 2013 at 1:34 PM, Ben Balter notifications@github.com
wrote:

Related posts sans LSI is just latest posts? Has that always been the case? Mind blown.

Reply to this email directly or view it on GitHub:
#1271 (comment)

@sanandnarayan sanandnarayan deleted the sanandnarayan:related_posts branch Jul 9, 2013
@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.