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

feed.xml: check to ensure the post excerpt isn't blank before outputting #57

Merged
merged 1 commit into from
Jun 18, 2015

Conversation

parkr
Copy link
Member

@parkr parkr commented Jun 15, 2015

Fixes #52

@parkr
Copy link
Member Author

parkr commented Jun 15, 2015

/cc @benbalter

@benbalter
Copy link
Contributor

and post.excerpt != blank

That works? Huh. Nice.

benbalter added a commit that referenced this pull request Jun 18, 2015
feed.xml: check to ensure the post excerpt isn't blank before outputting
@benbalter benbalter merged commit 6ad0ff1 into master Jun 18, 2015
@benbalter benbalter deleted the summary-be-empty branch June 18, 2015 05:19
@parkr
Copy link
Member Author

parkr commented Jun 18, 2015

Thanks, Ben!

@benbalter
Copy link
Contributor

@parkr is this tested anywhere? Looking at my own feed, it looks like this may have nuked excerpts entirely.

@parkr
Copy link
Member Author

parkr commented Aug 23, 2015

Looking at my own feed, it looks like this may have nuked excerpts entirely.

Hmmmmmm. Will look at your PR and see what's up.

@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
Development

Successfully merging this pull request may close these issues.

3 participants