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

Fix Truncate #881

Closed
wants to merge 1 commit into from
Closed

Fix Truncate #881

wants to merge 1 commit into from

Conversation

bep
Copy link
Member

@bep bep commented Feb 5, 2015

TruncateWordsToWholeSentence knows if the summary is truncated, so let "him" decide.

Fixes #880

TruncateWordsToWholeSentence knows if the summary is truncated, so let "him" decide.

Fixes gohugoio#880
@bep
Copy link
Member Author

bep commented Feb 5, 2015

@anthonyfok could you have a look at this and merge if you agree?. I have tested it and In my eyes this should fix it.

@anthonyfok
Copy link
Member

Brilliant, @bep! That does it! My local "Nitro" test page now correctly returns .Truncated as false! And I bet Hugo renders even a bit faster with this fix! Awesome!

And thank you so much for fixing it so quickly! I am deep in some paperwork + babysitting my little ones + sleep deprivation, not in the right mind to do any sort of debugging, and was thinking "let's leave it for v0.14", so your quick turnaround time is a pleasant surprise to me. Thank you!

@anthonyfok
Copy link
Member

Merged as 48b6777. Thanks again, @bep!

@anthonyfok anthonyfok closed this Feb 5, 2015
@bep bep deleted the fix/truncated branch April 18, 2017 09:19
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Need a new fix and test for page variable .Truncated
2 participants