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

hugolib: Fix .Site.LastChange #3058

Merged
merged 1 commit into from
Feb 19, 2017
Merged

hugolib: Fix .Site.LastChange #3058

merged 1 commit into from
Feb 19, 2017

Conversation

bep
Copy link
Member

@bep bep commented Feb 19, 2017

This commit makes sure that the .Site.LastChange is fetched from the latest page modification date.

Previously, this value was fetched from the last page in the default page sort, which may not be the last by date if weight is set.

Fixes #2909
Closes #2910

@bep
Copy link
Member Author

bep commented Feb 19, 2017

/cc @moorereason

@bep bep force-pushed the fix-lastchange branch 2 times, most recently from d1f5e0b to 1364119 Compare February 19, 2017 10:23
This commit makes sure that the `.Site.LastChange` is fetched from the latest page modification date.

Previously, this value was fetched from the last page in the default page sort, which may not be the last by date
if weight is set.

Fixes gohugoio#2909
Closes gohugoio#2910
@moorereason
Copy link
Contributor

Tested, and it works for me. 👍

@bep bep merged commit 6809ea1 into gohugoio:master Feb 19, 2017
@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 18, 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.

.Site.LastChange should be time of newest content
2 participants