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

Filter out drafts before limit #154

Merged
merged 2 commits into from
Jan 13, 2017
Merged

Conversation

pathawks
Copy link
Member

@pathawks pathawks commented Jan 5, 2017

It seems like drafts (which are not included in the feed) should not count toward the 10 post limit.

In a worst case scenario, having 10 or more drafts could result in an empty feed, no matter how many published posts the site contains.

@pathawks pathawks self-assigned this Jan 5, 2017
@pathawks
Copy link
Member Author

@jekyllbot: merge +bug

@jekyllbot jekyllbot merged commit 3c23143 into master Jan 13, 2017
@jekyllbot jekyllbot deleted the pull/filter-drafts-before-limit branch January 13, 2017 15:19
jekyllbot added a commit that referenced this pull request Jan 13, 2017
@jekyll jekyll locked and limited conversation to collaborators Apr 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants